Clean up the unused `build_in_import_modules` dictionary in ni-measurement-plugin-client-generator
还没有人认领这个 Issue。
评估
调研方向
打开 packages/generator/ni_measurement_plugin_sdk_generator/client/_support.py,检查未使用的 built_in_import_modules 字典和 _PATH_IMPORT 变量。删除两者,然后运行 generator 现有的检查或测试,以确认生成的 client 行为未发生变化,并且不再存在任何引用。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Python
- 星标
- 21
- 派生
- 20
- 平均合并
- 1 天 19 小时
- 30 天内合并 PR
- 2
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
ni/measurement-plugin-python 的其他 Issue
-
tech debt
难度 2/5 1-3 小时 新手友好度 65/100
ni/measurement-plugin-python#926 ·
-
AUTO Session Initialization Fails in Python Measurement Plugin with VISA Devices可能重新可做 @bkeryan 于 320 天前认领,目前没有进行中的 PR。 未关闭bug
ni/measurement-plugin-python#1263 · 1 条评论 · 已指派 1 人 ·
-
enhancement
难度 4/5 3-5 天 新手友好度 45/100
ni/measurement-plugin-python#1227 ·
-
tech debt
难度 3/5 1-2 天 新手友好度 45/100
ni/measurement-plugin-python#978 ·
-
tech debt
难度 3/5 1-2 天 新手友好度 48/100
ni/measurement-plugin-python#959 ·
查看 ni/measurement-plugin-python 的全部 Issue
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 90/100
维护者通常 1 天内回复
-
instance instance add
难度 2/5 1-3 小时 新手友好度 68/100
searxng/searx-instances#941 · 1 条评论 ·
-
难度 1/5 1 小时以内 新手友好度 92/100
FluidNumerics/fluid-walk-blocker#89 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复