Skip to content

Add changesets skill and update PR creation instructions#7825

Merged
jonrohan merged 2 commits into
mainfrom
add_changesets_skill
May 8, 2026
Merged

Add changesets skill and update PR creation instructions#7825
jonrohan merged 2 commits into
mainfrom
add_changesets_skill

Conversation

@jonrohan
Copy link
Copy Markdown
Member

@jonrohan jonrohan commented May 8, 2026

Adds a new Copilot skill for working with changesets and updates the Pull Request Creation section in copilot-instructions.md to require changesets (or the skip changeset label) for every PR.

New file: .github/skills/changesets/SKILL.md
A detailed skill covering:

  • What a changeset is and file format with examples
  • How to create one (CLI or manual file creation)
  • Semver bump quick-reference (patch/minor/major) with a link to contributor-docs/versioning.md
  • When changesets are NOT needed and the skip changeset label
  • Multiple changesets per PR guidance
  • Release pipeline context (changeset-bot, changesets/action)

Updated: .github/copilot-instructions.md
The ## Pull Request Creation section now has a numbered list requiring:

  1. The PR template
  2. A changeset file (when the change affects the published package)
  3. The skip changeset label (when no changeset is needed)

Changelog

New

  • .github/skills/changesets/SKILL.md — new Copilot skill for changeset guidance

Changed

  • .github/copilot-instructions.md — PR creation section updated to require changesets

Removed

Rollout strategy

  • None; if selected, include a brief description as to why

Documentation/instruction files only — no package code is changed and no release is needed.

Testing & Reviewing

Review the two files for accuracy and completeness:

  • .github/skills/changesets/SKILL.md — the new skill file
  • .github/copilot-instructions.md — the updated PR creation section (lines 155-161)

Merge checklist

Copilot AI review requested due to automatic review settings May 8, 2026 18:35
@jonrohan jonrohan requested a review from a team as a code owner May 8, 2026 18:35
@jonrohan jonrohan added the skip changeset This change does not need a changelog label May 8, 2026
@jonrohan jonrohan requested a review from joshblack May 8, 2026 18:35
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 8, 2026

⚠️ No Changeset found

Latest commit: 4d28d44

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions Bot added the staff Author is a staff member label May 8, 2026
@jonrohan jonrohan enabled auto-merge May 8, 2026 18:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds internal Copilot guidance for using Changesets in this repo, and updates Copilot PR-creation instructions so PRs include either a changeset file (when applicable) or the skip changeset label.

Changes:

  • Added a new Copilot skill doc: .github/skills/changesets/SKILL.md (what changesets are, how to create them, semver guidance, when to skip, release pipeline).
  • Updated .github/copilot-instructions.md to require PR authors to use the PR template and handle changesets consistently (add a changeset or apply skip changeset).
Show a summary per file
File Description
.github/skills/changesets/SKILL.md Introduces a Changesets skill doc for Copilot to reference when advising on versioning/release notes.
.github/copilot-instructions.md Updates PR creation instructions to require a changeset (or skip changeset label) in addition to using the PR template.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 4

Comment on lines +20 to +21
Changeset files live in `.changeset/` and have a random name with a `.md` extension. The file uses YAML frontmatter to specify the package and bump type, followed by a description:

- Remove a component
- Remove a prop
- Narrow the type of a prop (e.g., `string` → `'a' | 'b'`)
- Change the element that props are spread onto
Comment on lines +109 to +110
2. **On merge to `main`**: The `changesets/action` GitHub Action automatically creates or updates a "Version Packages" release PR that bumps version numbers, updates `CHANGELOG.md`, and shows release notes.
3. **On merging the release PR**: The action publishes the new version to npm and creates a GitHub Release.
When creating a pull request, you MUST:

1. Use the template in `.github/pull_request_template.md` to structure the PR description. Read the template file, fill in all sections appropriately, and include it in the PR body.
2. Include a changeset file in the PR if the change affects the published package (bug fix, new feature, breaking change, etc.). Create a `.changeset/<descriptive-name>.md` file with the appropriate semver bump type. Reference the `changesets` skill (`.github/skills/changesets/SKILL.md`) for detailed guidance on file format, choosing the correct bump type, and when a changeset can be skipped.
@jonrohan jonrohan added the integration-tests: skipped manually Changes in this PR do not require an integration test label May 8, 2026
@jonrohan jonrohan added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit 092df48 May 8, 2026
73 checks passed
@jonrohan jonrohan deleted the add_changesets_skill branch May 8, 2026 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: skipped manually Changes in this PR do not require an integration test skip changeset This change does not need a changelog staff Author is a staff member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants