Skip to content

Expanding list of CODEOWNERS for registry.json to include Foundry developers.#9164

Merged
richardpark-msft merged 2 commits into
mainfrom
azd-more-codeowners-for-registry
Jul 15, 2026
Merged

Expanding list of CODEOWNERS for registry.json to include Foundry developers.#9164
richardpark-msft merged 2 commits into
mainfrom
azd-more-codeowners-for-registry

Conversation

@richardpark-msft

@richardpark-msft richardpark-msft commented Jul 15, 2026

Copy link
Copy Markdown
Member

Welcome new developers to registry.json :) (once this PR merges!)

The basics of how this'll work for you, with regards to registry PRs:

  • If it's just a simple registry update (new version for an existing extension, no provider or capability updates) then you, or anyone else in CODEOWNERS can just approve and merge the PR.

  • If it's a complex one (ie, any of those things change), you'll see this PR workflow fail:

    image

    If you click it (link) it'll tell you the specific item that's requires approval. So then:

    1. Contact an azd core team member, and provided it's "okay" (@tg-msft is working on docs) then they'll approve it and the release can happen.
    2. Re-run that failed step and it'll evaluate and will pass.

Fixes #8997

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
22 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@richardpark-msft
richardpark-msft marked this pull request as ready for review July 15, 2026 22:15
Copilot AI review requested due to automatic review settings July 15, 2026 22:15
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
22 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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

Expands registry ownership so Foundry developers can approve routine extension publishing updates.

Changes:

  • Adds Foundry developers as code owners for registry.json.
  • Retains core-review enforcement for complex registry changes.
Show a summary per file
File Description
.github/CODEOWNERS Expands ownership of the extension registry.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Medium

@github-actions

Copy link
Copy Markdown

📋 Prioritization Note

Thanks for the contribution! The linked issue isn't in the current milestone yet.
Thank you for logging this issue; our team is reviewing it. If you need urgent prioritization, tag @RickWinter and @kristenwomack to let us know.

@richardpark-msft

Copy link
Copy Markdown
Member Author

/check-enforcer evaluate

@richardpark-msft

Copy link
Copy Markdown
Member Author

/check-enforcer override

@richardpark-msft
richardpark-msft merged commit 168f02a into main Jul 15, 2026
10 of 11 checks passed
@richardpark-msft
richardpark-msft deleted the azd-more-codeowners-for-registry branch July 15, 2026 22:27
@richardpark-msft

Copy link
Copy Markdown
Member Author

CC: @XiaofuHuang @achauhan-scc @anchenyi @glharper @huimiu @hund030 @kingernupur @saanikaguptamicrosoft @therealjohn @trangevi @trrwilson

Okay NOW you can approve extension release PRs, sorry about that false alarm earlier. :) Check the PR message for instructions, but it should be pretty much like any ole' PR for most situations.

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.

Auto-approve extension registry updates for version bump PRs

4 participants