已合并
chore: 精简未使用 Python 依赖(#166) #433
chore: 精简未使用 Python 依赖(#166) #433
已合并
sinat_31531339创建于 7月17日
1 个文件变更+1-6
@@ -1,15 +1,10 @@
1pytest>=9.0.21pytest>=9.0.2
zhangjiezhangjie
zhangjiezhangjie7月18日

级别:严重 问题:移除了 PyYAML 依赖,但若项目中存在 import yaml 的代码,将导致运行时 ImportError。 影响:任何使用 YAML 配置文件解析或序列化的功能将在运行时崩溃。 修复建议:在合并前全局搜索 import yamlyaml. 确认无引用;若仍有使用,请保留 PyYAML 依赖。

likedislike
zhangjiezhangjie7月18日

级别:严重 问题:移除了 requests 依赖,但若项目中存在 import requests 的代码,将导致运行时 ImportError。 影响:任何使用 requests 进行 HTTP 请求的功能(如下载模型、API 调用)将在运行时崩溃。 修复建议:在合并前全局搜索 import requests 确认无引用;若仍有使用,请保留 requests 依赖或替换为其他 HTTP 库。

likedislike
zhangjiezhangjie7月18日

级别:一般 问题:移除了 pytest-cov,但保留了 coverage。若 CI 脚本或 Makefile 中使用 pytest --cov 参数,将导致覆盖率收集失败。 影响:CI 流水线中覆盖率收集步骤可能报错退出,阻断流水线。 修复建议:检查 CI 配置和测试脚本中是否使用 --cov 参数;若使用,需保留 pytest-cov 或改为直接使用 coverage run -m pytest 方式。

likedislike
zhangjiezhangjie7月18日

级别:一般 问题:移除了 ruff 和 yapf,若 CI 中有代码风格检查或格式化步骤,将导致 CI 失败。 影响:代码质量门禁可能被绕过或 CI 步骤报错。 修复建议:检查 CI 配置中是否调用 ruff/yapf;若有,需将它们移至单独的 dev-requirements 或 lint-requirements 文件中,而非直接删除。

likedislike
zhangjiezhangjie7月18日

级别:提示 问题:移除多个依赖后未说明变更原因,缺乏可追溯性。 影响:后续维护者难以判断这些依赖是否可安全移除,可能引入回归。 修复建议:在 PR 描述中补充移除每个依赖的理由,并附上全局搜索无引用的证据。

likedislike
2-pytest-cov>=4.0.0
3coverage>=7.13.22coverage>=7.13.2
4pytest-mock>=3.15.13pytest-mock>=3.15.1
5numpy>=1.26.44numpy>=1.26.4
6pandas>=2.2.0,<2.45pandas>=2.2.0,<2.4
7protobuf>=6.33.46protobuf>=6.33.4
8setuptools>=70.1.07setuptools>=70.1.0
9-ruff>=0.14.14
10-yapf>=0.43.0
11-PyYAML>=6.0.3
12-requests>=2.32.5
13h5py>=3.10.08h5py>=3.10.0
14psutil>=7.2.29psutil>=7.2.2
15-decorator>=4.4.210+decorator>=4.4.2