Skip to content

Remove dev_experience from theme and changeset CODEOWNERS#8024

Open
karreiro wants to merge 1 commit into
mainfrom
update-codeowners-file-themes
Open

Remove dev_experience from theme and changeset CODEOWNERS#8024
karreiro wants to merge 1 commit into
mainfrom
update-codeowners-file-themes

Conversation

@karreiro

@karreiro karreiro commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

PRs like #6817 are unrelated to the @Shopify/dev_experience / Apps domain, but our CODEOWNERS setup keeps creating friction for both github teams by requesting reviews that don’t add much value.

We already have guardrails in place, like #6803 and the bot reminders around changeset updates, so it feels like we’ve got enough coverage to ensure changesets keep working as intended.

This PR updates the CODEOWNERS file to reduce unnecessary friction on review requests and make the code review process leaner for the Themes domain.

@karreiro karreiro requested a review from a team as a code owner July 7, 2026 13:45
Copilot AI review requested due to automatic review settings July 7, 2026 13:45
@karreiro karreiro requested a review from a team as a code owner July 7, 2026 13:45
@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jul 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts GitHub CODEOWNERS to reduce unnecessary review requests from @shopify/dev_experience for Themes-related changes and for changeset files, aiming to streamline reviews for the Themes domain.

Changes:

  • Removed @shopify/dev_experience as a CODEOWNER for theme-related paths under packages/cli-kit/.../themes/* and packages/theme/**.
  • Attempted to make .changeset/* reviewable without explicit owners (but the current line is syntactically invalid in CODEOWNERS and won’t achieve the intended effect).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/CODEOWNERS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants