ni / ni/measurement-plugin-python
Clean up the unused `build_in_import_modules` dictionary in ni-measurement-plugin-client-generator
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 21
- Forks
- 20
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 3
Description
Tech Debt
As discussed in this comment.
In #871, we hard-coded the 'import pathlib', but continued iterating through the built_in_import_modules dictionary, which only contained the 'import pathlib' string literal until it was removed in #900. The use of built_in_import_modules was eliminated in #931. As a result, the built_in_import_modules dictionary and the _PATH_IMPORT variable are no longer used or required and should be removed.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open packages/generator/ni_measurement_plugin_sdk_generator/client/_support.py and inspect the unused built_in_import_modules dictionary and _PATH_IMPORT variable. Remove both, then run the generator's existing checks or tests to confirm generated client behavior is unchanged and no references remain.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 68/100