Skip to content

Commit

Permalink
Update CODEOWNERS paths: fix invalid paths
Browse files Browse the repository at this point in the history
As part of ongoing work of enabling wildcard support for `CODEOWNERS`:
- Azure/azure-sdk-tools#2770
- Azure/azure-sdk-tools#5088

and enabling stricter validation:
- Azure/azure-sdk-tools#4859

this PR:
- fixes invalid paths, to match rules explained [here](https://github.com/Azure/azure-sdk/blob/main/docs/policies/opensource.md#codeowners);
- removes `/**/tests.yml` and `/**/ci.yml`, to avoid all build failure notifications being routed to it once we enable the new regex-based, wildcard-supporting `CODEOWNERS` matcher, per: Azure/azure-sdk-tools#5088 (comment)

Once this PR is merged, I will enable the new `CODEOWNERS` matcher, similar to how it was done for `net` repo by these two PRs:
- Azure/azure-sdk-tools#5241
- Azure/azure-sdk-tools#5240
  • Loading branch information
Konrad Jamrozik authored Feb 10, 2023
1 parent ced0955 commit 498725c
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,6 +1,17 @@
# Instructions for CODEOWNERS file format and automatic build failure notifications:
# https://github.com/Azure/azure-sdk/blob/main/docs/policies/opensource.md#codeowners

################
# Orphaned paths
################
# As of 2/9/2023 these paths have no owners:

# /**
# /.run/
# /.vscode/
# /config/
# /doc/

################
# Automation
################
Expand Down Expand Up @@ -36,5 +47,3 @@
# Eng Sys
###########
/eng/ @benbp @weshaggard

/**/ci.yml @benbp

0 comments on commit 498725c

Please sign in to comment.