Skip to content

Add checklist to PR template#231

Merged
Thaleszh merged 1 commit into
mainfrom
docs/bump-pr-template
Jan 14, 2026
Merged

Add checklist to PR template#231
Thaleszh merged 1 commit into
mainfrom
docs/bump-pr-template

Conversation

@Thaleszh

@Thaleszh Thaleszh commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

Description

Start a small checklist for PR changes

Type of change

  • Documentation (updates documentation on the project)

@Thaleszh Thaleszh requested a review from jhelison as a code owner January 14, 2026 14:34
@coderabbitai

coderabbitai Bot commented Jan 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Walkthrough

The pull request adds a "PR Checklist" section to the GitHub pull request template. The new section introduces two checklist items requiring developers to update the changelog with the PR's intent and run linting with make lint-fix. The modification preserves existing test-related checklist items and consists of 7 added lines with no removals.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: adding a checklist section to the PR template.
Description check ✅ Passed The description is related to the changeset, explaining the intent to add a small checklist for PR changes and marking it as a documentation update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f14fe42 and 3661e12.

📒 Files selected for processing (1)
  • .github/PULL_REQUEST_TEMPLATE.md
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: repo-analysis
🔇 Additional comments (1)
.github/PULL_REQUEST_TEMPLATE.md (1)

22-27: No changes needed — the referenced resources exist.

Verification confirms that both CHANGELOG.md and the make lint-fix target exist in the repository, so the checklist items appropriately reference real resources that contributors should already be working with.

Likely an incorrect or invalid review comment.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Thaleszh Thaleszh merged commit 27b4e79 into main Jan 14, 2026
6 of 7 checks passed
@Thaleszh Thaleszh deleted the docs/bump-pr-template branch January 14, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jhelison jhelison jhelison approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Thaleszh @jhelison