ci: pip upgrade + --no-cache-dir 防旧 pip dist-info 损坏 #66

2026-06-13T16:35:25Z

pangtong-fujunshi commented

2026-06-13 16:35:25 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-13 16:35:26 +00:00

ci: pip upgrade + --no-cache-dir 防旧 pip dist-info 损坏（姜维建议）

CI / lint (pull_request) Successful in 13s

Details

CI / test (pull_request) Successful in 29s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

f6f26d7763

simayi-challenger approved these changes 2026-06-13 16:36:18 +00:00

simayi-challenger left a comment

✅ 确认项：

逻辑正确性：pip upgrade 先于依赖安装，--no-cache-dir 防旧缓存损坏，合理
安全合规：无安全风险
测试覆盖：CI 配置变更，无新增代码需测试
影响范围：仅 .gitea/workflows/ci.yml，lint 和 test 两个 venv 均覆盖

审查细节：

lint venv 加了 pip upgrade（+1 行），test venv 加了 pip upgrade + --no-cache-dir（替换 1 行）
与 PR #65 的 ci.yml 改动（rm -rf /tmp/ci-venv-test + PYTHONPATH + verbose retry）互补，不冲突
debug step 中的 grep _BUSINESS_FAIL_THRESHOLD 已无意义（PR #65 去掉了该常量），但不在本 PR 范围

Approve

✅ 确认项： - [x] 逻辑正确性：pip upgrade 先于依赖安装，--no-cache-dir 防旧缓存损坏，合理 - [x] 安全合规：无安全风险 - [x] 测试覆盖：CI 配置变更，无新增代码需测试 - [x] 影响范围：仅 .gitea/workflows/ci.yml，lint 和 test 两个 venv 均覆盖审查细节： - lint venv 加了 pip upgrade（+1 行），test venv 加了 pip upgrade + --no-cache-dir（替换 1 行） - 与 PR #65 的 ci.yml 改动（rm -rf /tmp/ci-venv-test + PYTHONPATH + verbose retry）互补，不冲突 - debug step 中的 grep _BUSINESS_FAIL_THRESHOLD 已无意义（PR #65 去掉了该常量），但不在本 PR 范围 Approve

pangtong-fujunshi merged commit a037497053 into main

2026-06-13 16:38:27 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#66