Skip to content

Adjust copilot instructions#1339

Merged
JonasDov merged 1 commit into
masterfrom
JonasD/copilot-instructions-fix
May 7, 2026
Merged

Adjust copilot instructions#1339
JonasDov merged 1 commit into
masterfrom
JonasD/copilot-instructions-fix

Conversation

@JonasDov
Copy link
Copy Markdown
Contributor

@JonasDov JonasDov commented May 7, 2026

Changes due to this comment: #1338 (comment)

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings May 7, 2026 08:16
@JonasDov JonasDov requested a review from a team as a code owner May 7, 2026 08:16
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 7, 2026

⚠️ No Changeset found

Latest commit: 71237e5

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 changesets to release 1 package
Name Type
@itwin/presentation-components Patch

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

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

Updates the repository’s Copilot PR-review guidance around changeset wording, and aligns an existing changeset description with the clarified convention (no component prefix for package-wide dependency bumps).

Changes:

  • Clarify changeset description guidance to only prefix with a component name when the change is scoped to a single component/class, not for package-wide dependency bumps.
  • Update an existing changeset description to remove the unnecessary “Presentation Components:” prefix for an AppUI dependency bump.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/copilot-instructions.md Clarifies when changeset descriptions should (and should not) be prefixed with a component name.
.changeset/bump-appui-dependencies.md Updates changeset text to match the clarified “no prefix for dependency bumps” instruction.

@JonasDov JonasDov enabled auto-merge (squash) May 7, 2026 08:19
@JonasDov JonasDov merged commit adc250f into master May 7, 2026
19 checks passed
@JonasDov JonasDov deleted the JonasD/copilot-instructions-fix branch May 7, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants