chore: update issue & PR templates#6673
Conversation
Signed-off-by: Dwi Siswanto <git@dw1.io>
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughA new "### Proof" section was added to the pull request template, providing a placeholder prompt for contributors to document how their changes were tested. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.github/PULL_REQUEST_TEMPLATE.md (1)
5-7: Consider a more conventional section name.The new "### Proof" section is a helpful addition that encourages contributors to document their testing approach. However, the section name is somewhat informal. Consider more conventional alternatives like "### Testing" or "### How to Test" for better clarity and consistency with industry-standard PR template conventions.
Apply this diff if you'd like to use a more standard naming:
-### Proof +### TestingThe guidance prompt itself is clear and actionable—this is a minor naming preference that can be deferred if your team prefers "Proof" as the convention.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
.github/ISSUE_TEMPLATE/bug-report.ymlis excluded by!**/*.yml
📒 Files selected for processing (1)
.github/PULL_REQUEST_TEMPLATE.md(1 hunks)
Signed-off-by: Dwi Siswanto <git@dw1.io>
Proposed changes
Checklist
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.