chore: remove redundant PR checklist template - #102
Conversation
All three checklist items are already enforced by CI in .github/workflows/lint.yml (commit-check for conventional commits, prek for pre-commit/pixi run lint); the third item was tautological. Assisted-by: claude-code:claude-opus-4-8[1m]
|
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 (1)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThe pull request template no longer includes the checklist for commit messages, pre-commit lint checks, and CI checks. Changes
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This change removes a redundant checklist without affecting product behavior or runtime operation; no actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
Physics Metrics ComparisonAll configurations match reference (no significant differences)
About this comparisonPhysics metrics are extracted from ROOT files and compared. Metrics stored in git notes: |
🤖 AI text below 🤖
The PR template contained only a three-item checklist, all of which are already enforced automatically by CI in
.github/workflows/lint.yml:commit-checkjobpixi run lint) →prekjobSince the checklist was the entire file, this removes
.github/pull_request_template.md. GitHub falls back to a blank PR description box.Summary by CodeRabbit