fix(build): bind Maven Wrapper distribution to reviewed SHA-256 - #319
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughMaven Wrapper 설정에 Maven 배포 ZIP의 SHA-256 체크섬을 추가했습니다. JUnit 테스트는 Wrapper 버전, 배포 유형, 고정 URL, 체크섬 형식과 값을 검증합니다. ChangesMaven Wrapper 무결성
Estimated code review effort: 2 (Simple) | ~10 minutes Mergeability Score: ⚪ Minimal · up to This localized change binds the Maven Wrapper distribution to the reviewed SHA-256 without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. Possibly related PRs
🚥 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 |
Implements issue #265 from exact protected base
develop@8f96517987cf2cab4398696503b22a131a4b66a6without rewriting TDD history.TDD lineage:
2206afe476189856759bd3cedd989debf697eb9f— test-only RED requiring the fixed Maven 3.9.11 distribution URL to carry its reviewed SHA-256.717945e15ca944d6592d77f2ab504cb9da9e8e3a— smallest GREEN: addsdistributionSha256Sum=0d7125e8c91097b36edb990ea5934e6c68b4440eef4ea96510a0f6815e7eeadbwhile preserving Wrapper 3.3.4,only-script, Maven 3.9.11, and the distribution URL.Current-head hosted aggregates are terminal-success: CI
31690169809, Dependency Review31690169850, CycloneDX SBOM31690169804, SAST Semgrep31690169845, and Security Scan31690169851. These are current PR evidence only; synthetic-merge execution, scanner-completeness, repository-wide coverage, and independent-review requirements remain separate gates and no predecessor-head evidence transfers.CodeRabbit skipped while Draft. Marking ready should trigger the ordinary review path; merge still requires unchanged exact head, zero valid unresolved findings, qualifying independent non-author formal approval, and all live governance gates.
Summary by CodeRabbit