prepare-pr

name	prepare-pr
description	Prepare, open, create, publish, update, or edit a NeMo Fabric pull request or PR body with the right tests, docs, scope, and review handoff details
author	NVIDIA Corporation and Affiliates
license	Apache-2.0

Prepare A PR For NeMo Fabric

Companion Guidance

Use karpathy-guidelines alongside this skill for implementation or review work. Keep changes scoped, surface assumptions, and define focused validation before editing.

Use this skill at the end of a contributor or maintainer change before opening a pull request. Also use it whenever a user asks to create, open, publish, update, or edit a NeMo Fabric pull request, pull request description, or PR body.

If this repo-local guidance conflicts with generic GitHub publishing, connector, or plugin guidance, this skill wins for PR body format, validation language, and review handoff details.

Checklist

• Branch scope is coherent and reviewable
• Relevant tests passed under validate-change
• Changed files were formatted with the language-native formatter
• Any Rust change ran just test-rust
• Any Rust change ran cargo fmt --all -- --check
• Native binding changes ran cargo check -p fabric-python --locked
• crates/fabric-core changes ran both the Rust and Python suites
• Docs and examples updated for any public behavior changes
• Dependent maintainer or consumer skills updated when code changes affected their APIs, bindings, commands, paths, packaging guidance, or best practices
• New or updated dependencies include the functional need, alternatives considered, and why the selected dependency is the narrowest fit
• The lockfile license diff was reviewed for direct and transitive changes; unresolved, custom, or copyleft terms are called out for dependency approver review
• Changed ATTRIBUTIONS-*.md files are regenerated and included
• Pull request title follows Conventional Commit style and uses the correct type
• Pull request body follows the repository template when one exists
• Breaking changes or renamed surfaces are called out explicitly

Pull Request Title

Use Conventional Commit style for PR titles:

<type>: <concise imperative summary>

Choose the type from the actual change surface, not from the impact of the review comment or CI outcome. Use fix only for an actual user-facing or runtime/product code bug fix. Never use fix for changes that are not related to product code behavior, including chores, CI configuration, docs, tests, packaging metadata, generated-output handling, or agent/skill guidance.

Common examples:

• ci: use just recipes in Rust checks
• docs: clarify adapter setup
• chore: refresh generated API references
• test: add session regression coverage
• fix: preserve runtime session identity

Opening A Pull Request

Always use .github/pull_request_template.md as the source of truth for the PR body. Before opening a PR, read the current template and preserve its headings, checkboxes, comments' intent, and related-issue guidance.

This applies both when creating a new PR and when editing an existing PR description. Do not use a generic Summary / Why / Validation body unless the current repository template uses those headings.

When using GitHub CLI, prefer:

gh pr create --template .github/pull_request_template.md

If a tool cannot consume the template directly, create the PR body from the template content and then fill in every visible section before opening the PR. Do not replace the template with a freeform summary.

After creating or editing a PR, fetch the rendered PR body and verify that the template's visible headings and checklist items are still present.

The PR body must include:

• #### Overview with a concise summary and both contribution confirmation checklist items preserved
• #### Details with the concrete changes made
• #### Validation with commands run and any checks not run
• #### Where should the reviewer start? with the most useful file, test, or design decision
• #### Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to) with an issue reference, or a clear Relates to: none entry when there is no related issue

For dependency changes, include the dependency rationale and material license diff findings in the template's #### Overview. The license diff is review evidence, not a compatibility decision.

Only check the contribution confirmation boxes when they are true. If either confirmation cannot be made, stop before opening the PR and surface the blocker.

References

• README.md
• .github/pull_request_template.md
• maintain-packaging
• scripts/licensing/license_diff.py
• validate-change