xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 9 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

fix(ci): 修复check_migration_safety.py的import排序(isort I001) #371

Merged
xiaoxia merged 1 commits from fix/isort-import-sorting into develop 2026-07-15 22:57:07 +08:00
Conversation 1 Commits 1 Files Changed 1
+1 -1
xiaoxia commented 2026-07-15 18:12:54 +08:00
Owner
Copy Link
Copy Source

问题根因

push模式下Validate全量扫描失败，43秒报错。

根因：scripts/check_migration_safety.py中urllib.request和urllib.error的import顺序不符合isort规范。

PR模式下是增量扫描，这个文件没被修改所以没触发；push到develop是全量扫描暴露。

修复

  • 交换urllib.error和urllib.request顺序，符合isort字母排序

验证

  • ruff check --select I scripts/check_migration_safety.py 通过✅
## 问题根因 push模式下Validate全量扫描失败，43秒报错。 根因：`scripts/check_migration_safety.py`中`urllib.request`和`urllib.error`的import顺序不符合isort规范。 PR模式下是增量扫描，这个文件没被修改所以没触发；push到develop是全量扫描暴露。 ## 修复 - 交换`urllib.error`和`urllib.request`顺序，符合isort字母排序 ## 验证 - `ruff check --select I scripts/check_migration_safety.py` 通过✅
xiaoxia added 1 commit 2026-07-15 18:12:54 +08:00
fix(ci): 修复check_migration_safety.py的import排序(isort I001)
CI/CD Pipeline / Frontend Lint (push) Successful in 37s
Details
CI/CD Pipeline / Build Production API Image (push) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (push) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (push) Has been skipped
Details
CI/CD Pipeline / Deploy Production (push) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (push) Has been skipped
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 40s
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Validate Code Quality And Tests (push) Successful in 1m47s
Details
CI/CD Pipeline / Build Staging API Image (push) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (push) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (push) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (push) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (push) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (push) Has been skipped
Details
CI/CD Pipeline / Unit Tests (push) Successful in 1m58s
Details
CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Successful in 1m31s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 1m46s
Details
CI/CD Pipeline / Integration Tests (push) Successful in 1m15s
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 1m12s
Details
d5a8d9790b
auto-approve-bot approved these changes 2026-07-15 18:22:31 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
xiaoxia merged commit dad02788e3 into develop 2026-07-15 22:57:07 +08:00
xiaoxia deleted branch fix/isort-import-sorting 2026-07-15 22:57:07 +08:00
Sign in to join this conversation.
Reviewers
No Reviewers
auto-approve-bot
Labels
Clear labels
P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot xiaoxia
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#371
Delete Branch "fix/isort-import-sorting"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?