Skip to content

Update CODEOWNERS for azure.ai.agents#7722

Open
trangevi wants to merge 1 commit intomainfrom
trangevi/codeowners-update
Open

Update CODEOWNERS for azure.ai.agents#7722
trangevi wants to merge 1 commit intomainfrom
trangevi/codeowners-update

Conversation

@trangevi
Copy link
Copy Markdown
Member

Adjust agents extension code owners to specifically be the agents team

Adjust agents extension code owners to specifically be the agents team
@trangevi trangevi requested review from a team and rajeshkamal5050 as code owners April 14, 2026 20:40
Copilot AI review requested due to automatic review settings April 14, 2026 20:40
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

Note

Copilot was unable to run its full agentic suite in this review.

Updates CODEOWNERS so that the azure.ai.agents CLI extension is owned specifically by the agents team rather than the broader AI extension owner set.

Changes:

  • Narrowed the CODEOWNERS entry for /cli/azd/extensions/azure.ai.agents/ to a smaller, agents-focused owner group.

Comment thread .github/CODEOWNERS
@trangevi trangevi linked an issue Apr 14, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

📋 Prioritization Note

Thanks for the contribution! The linked issue isn't in the current milestone yet.
Review may take a bit longer — reach out to @rajeshkamal5050 or @kristenwomack if you'd like to discuss prioritization.

Comment thread .github/CODEOWNERS
Comment thread .github/CODEOWNERS

# ── CLI extensions (AI) ──────────────────────────────────────────────────────
/cli/azd/extensions/azure.ai.agents/ @jongio @wbreza @vhvb1989 @hemarina @weikanglim @JeffreyCA @tg-msft @rajeshkamal5050 @trangevi @trrwilson @therealjohn
/cli/azd/extensions/azure.ai.agents/ @trangevi @trrwilson @therealjohn @glharper
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I prefer to keep azd folks on this and add foundry folks instead of remove azd

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

My main concern is about broader directional understanding for the extension. Code review from the azd team is valuable and desirable, but as more and more things get added, I would assume that it won't be your role to make sure that changes are following the desired direction for the extension, more so for azd all up. So I'm concerned that a change will be proposed, reviewed and approved by azd folks, and merged, before foundry folks are even aware of it (I already saw it with one of Glenn's PRs, which got me thinking of this).

Since GH doesn't have a concept of required and optional reviewers, I couldn't think of another way to address the concern other than removing azd folks as codeowners, and instead manually requesting reviews from you, while leaving foundry folks as the final sign off. If there's a better way to accomplish this I'm very much open to it, as I definitely don't want to lose the current collaboration.

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.

Adjust codeowners for agents extension

4 participants