Skip to content

chore: add azure-app-onboard(-prereq) code owner - #3020

Merged
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/app-onboard-code-owner
Aug 6, 2026
Merged

chore: add azure-app-onboard(-prereq) code owner#3020
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/app-onboard-code-owner

Conversation

@JasonYeMSFT

@JasonYeMSFT JasonYeMSFT (JasonYeMSFT) commented Aug 5, 2026

Copy link
Copy Markdown
Member

Description

Add azure-app-onboard and azure-app-onboard-prereq code owners.

Checklist

  • Tests pass locally (cd tests && npm test)
  • Title has one of the prefixes: fix:, feat:, feature:, chore:, misc:, test:, eval:
  • If modifying skill descriptions: verified routing correctness with integration tests (In tests/, npm run test:integration -- <skill> or npm run test:vally -- --skill <skill>)

Related Issues

fixes #3016

@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) requested a review from a team as a code owner August 5, 2026 17:25
Copilot AI lite review requested due to automatic review settings August 5, 2026 17:25

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

Adds CODEOWNERS entries so changes under the two Azure onboarding skill directories automatically request review from the appropriate maintainers.

Changes:

  • Added code owner mapping for /plugins/azure-skills/skills/azure-app-onboard/
  • Added code owner mapping for /plugins/azure-skills/skills/azure-app-onboard-prereq/
Show a summary per file
File Description
.github/CODEOWNERS Adds ownership rules for the azure-app-onboard and azure-app-onboard-prereq skill directories.

Review details

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

msalaman
msalaman previously approved these changes Aug 6, 2026

@msalaman msalaman 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.

Looks good but merge conflict needs to be resolved

@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) merged commit aeb1b11 into microsoft:main Aug 6, 2026
9 checks passed
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.

CODEOWNERS for skill azure-app-* is missing

3 participants