test(actions): remove retired steward contract - #145
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough시간별 PR 스튜어드의 정적 계약 테스트 파일 전체를 삭제했습니다. 관련 헬퍼와 다섯 개의 테스트 함수도 함께 제거했습니다. Changes
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This removes the stale test for a retired workflow, eliminating its file-not-found failures without changing centralized workflow ownership or runtime behavior. No merge-blocking risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
이 삭제는 #146의 현재 head 에 포함되어 전체 중앙 dispatch 및 orchestrator/free 전환과 함께 검증됩니다. 중복 Checks를 줄이기 위해 superseded로 닫습니다. |
|
정정: 삭제 내용은 #146의 현재 head |
원인
중앙 required workflows로 PR 운영을 통합한 #144가 repo-local
.github/workflows/hourly-pr-steward.yml을 제거했지만, 삭제된 workflow의 정적 계약 테스트는main에 남았습니다. 그 결과 unrelated PR의 merge ref에서도 account-unification suite가 5건의FileNotFoundError로 실패했습니다.수정
목적을 마친 self-modifying workflow의 잔여 테스트 한 파일만 제거했습니다. workflow를 복원하거나 중앙 owner 경계를 우회하지 않습니다.
검증
100917dba555531bdaad0db3780da80686e237d8239e362c95d48894a10841ec8a087f9107f3f90cuv run --python 3.12 pytest -q services/account_unification/tests통과git diff --check통과이 PR이 보호 병합된 뒤 Keyverse #128을 non-force로 최신 main에 restack하고 exact-head 검증을 다시 받습니다.
Summary by CodeRabbit