Skip to content

chore(codeowners): add missing directory paths#256

Open
WilliamBerryiii wants to merge 1 commit intomainfrom
chore/update-codeowners-paths-135
Open

chore(codeowners): add missing directory paths#256
WilliamBerryiii wants to merge 1 commit intomainfrom
chore/update-codeowners-paths-135

Conversation

@WilliamBerryiii
Copy link
Member

Description

Added five missing directory paths to CODEOWNERS, ensuring @microsoft/edge-ai-core-dev is automatically requested for reviews when changes touch these previously unowned directories. The change was purely additive — no existing paths were removed or reordered.

The additions filled coverage gaps across four sections:

  • CI/CD pipelines — added /.azuredevops/ and /deploy/ alongside existing /.azdo/ and /azure-pipelines.yml entries.
  • Documentation — added /project-adrs/ after the existing /learning/ entry.
  • Development environment — added /.vscode/ after the existing /.devcontainer/ entry.
  • Static assets — introduced a new section with its own comment header and added /assets/.

All five paths were assigned to @microsoft/edge-ai-core-dev, consistent with every other entry in the file.

Related Issue

Related to #135

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Blueprint modification or addition
  • Component modification or addition
  • Documentation update
  • CI/CD pipeline change
  • Other (please describe): Repository governance — expanded CODEOWNERS coverage for unowned directories

Implementation Details

Added five directory paths to .github/CODEOWNERS in a single commit (179f18cd6), organized within the file's existing sectioned-comment convention:

  • Inserted /.azuredevops/ and /deploy/ under the CI/CD pipelines comment block.
  • Inserted /project-adrs/ under the Documentation comment block.
  • Inserted /.vscode/ under the Development environment comment block.
  • Created a new # Static assets comment header and added /assets/ beneath it.

All entries use leading-slash root-relative format, matching the established pattern throughout the file.

Testing Performed

  • Terraform plan/apply
  • Blueprint deployment test
  • Unit tests
  • Integration tests
  • Bug fix includes regression test (see Test Policy)
  • Manual validation
  • Other:

Manual validation confirmed all five paths are present and correctly placed in the file, following the repository's sectioned-comment convention and root-relative path format.

Validation Steps

  1. Open .github/CODEOWNERS and verify the five new entries exist under their respective section headers.
  2. Confirm each path uses leading-slash root-relative format (e.g., /.azuredevops/).
  3. Confirm all entries are assigned to @microsoft/edge-ai-core-dev.
  4. Verify the new # Static assets section header follows the existing comment convention.

Checklist

  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • I have run terraform fmt on all Terraform code
  • I have run terraform validate on all Terraform code
  • I have run az bicep format on all Bicep code
  • I have run az bicep build to validate all Bicep code
  • I have checked for any sensitive data/tokens that should not be committed
  • I have run MegaLinter on my code (mega-linter-runner)

Additional Notes

This change is purely additive to .github/CODEOWNERS — no Terraform, Bicep, or application code was modified. The Terraform, Bicep, and MegaLinter checklist items are not applicable to this change.

Screenshots (if applicable)

Not applicable.

- Add /deploy/ under CI/CD pipelines
- Add /.azuredevops/ under CI/CD pipelines
- Add /project-adrs/ under Documentation
- Add /.vscode/ under Development environment
- Add /assets/ under Static assets

📋 - Generated by Copilot
@WilliamBerryiii WilliamBerryiii requested a review from a team as a code owner March 15, 2026 00:40
Copilot AI review requested due to automatic review settings March 15, 2026 00:40
Copy link
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 repository ownership rules so the core team is explicitly requested for review on changes to additional CI/CD, tooling, and documentation directories.

Changes:

  • Add CODEOWNERS entries for .azuredevops/ and deploy/ (CI/CD-related content).
  • Add CODEOWNERS entries for project-adrs/, .vscode/, and assets/.

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.

2 participants