Skip to content

Conversation

@ogenstad
Copy link
Contributor

@ogenstad ogenstad commented Feb 10, 2026

Adds the same pull request template as we have in the Infrahub project.

Summary by CodeRabbit

  • Chores
    • Added a pull request template to guide contributors in documenting changes with structured sections (Why, What changed, How to review, How to test, Impact & rollout, Checklist). The template clarifies it's a guide (not a gate) and provides placeholders and guidance to improve consistency, review quality, and easier validation of proposed changes.

@github-actions github-actions bot added the group/ci Issue related to the CI pipeline label Feb 10, 2026
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 10, 2026

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3c7fef2
Status: ✅  Deploy successful!
Preview URL: https://dbfadbda.infrahub-sdk-python.pages.dev
Branch Preview URL: https://pog-add-pull-request-templat.infrahub-sdk-python.pages.dev

View logs

@ogenstad ogenstad marked this pull request as ready for review February 10, 2026 13:13
@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

Walkthrough

A new pull request template was added at .github/pull_request_template.md. The Markdown file defines sections for Why, What changed, How to review, How to test, Impact & rollout, and a Checklist, with guidance text and placeholders for PR descriptions and validation steps. The change only updates repository configuration; no executable code or public API declarations were modified.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Add pull request template' directly and clearly describes the main change in the changeset, which is the addition of a new pull request template file.
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.


No actionable comments were generated in the recent review. 🎉


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.github/pull_request_template.md:
- Line 70: Update the broken relative link in the PR template checklist: replace
the non-existent path "../dev/guidelines/changelog.md" with the repository root
changelog "CHANGELOG.md" in the checkbox line (the checklist item containing
"Changelog entry" in .github/pull_request_template.md) so the link points to an
existing file.

@ogenstad ogenstad force-pushed the pog-add-pull-request-template branch from 080bb44 to 3c7fef2 Compare February 10, 2026 13:33
@ogenstad ogenstad requested a review from a team February 10, 2026 13:44
@ogenstad ogenstad merged commit 497bad1 into stable Feb 10, 2026
15 checks passed
@ogenstad ogenstad deleted the pog-add-pull-request-template branch February 10, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-code-assisted group/ci Issue related to the CI pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants