chore: restructure of the PR template - #37
Conversation
Signed-off-by: David Gardner <dagardner@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📜 Recent review details🔇 Additional comments (1)
WalkthroughThe pull request template was updated to replace the opening description guidance and details section with an HTML comment prompt, and to move two checklist items later in the template after the related-issues placeholder. ChangesPR Template Update
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/pull_request_template.md:
- Line 14: The pull request template is missing a final trailing newline, which
keeps markdownlint warnings active. Update the .github pull_request_template.md
content so it ends with exactly one newline after the last line; this is a
formatting-only fix and should not change any checklist text or structure.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Enterprise
Run ID: fc30c105-41e2-459e-94eb-7c1a3241d56c
📒 Files selected for processing (1)
.github/pull_request_template.md
📜 Review details
🧰 Additional context used
🪛 markdownlint-cli2 (0.22.1)
.github/pull_request_template.md
[warning] 14-14: Files should end with a single newline character
(MD047, single-trailing-newline)
🔇 Additional comments (1)
.github/pull_request_template.md (1)
3-4: LGTM!
|
📖 Fern docs preview: https://nvidia-preview-pull-request-37.docs.buildwithfern.com/nemo/fabric |
Signed-off-by: David Gardner <dagardner@nvidia.com>
Overview
Summary by CodeRabbit