Skip to content

Commit 4f7ac15

Browse files
Konrad Jamrozikbenbp
andauthored
Update CODEOWNERS paths: fix invalid paths (Azure#19994)
* Update `CODEOWNERS` paths: fix invalid paths 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 Related PRs: - Similar PR fixing invalid paths, but for `net` repo: Azure/azure-sdk-for-net#33584 - Similar PR deprioritizing the Azure SDK EngSys team ownership, but for `python` repo: Azure/azure-sdk-for-python#28534 * Update .github/CODEOWNERS Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> --------- Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
1 parent acf9938 commit 4f7ac15

File tree

1 file changed

+10
-15
lines changed

1 file changed

+10
-15
lines changed

.github/CODEOWNERS

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,12 @@
11
# Instructions for CODEOWNERS file format and automatic build failure notifications:
22
# https://github.com/Azure/azure-sdk/blob/master/docs/policies/opensource.md#codeowners
33

4-
################
5-
# Automation
6-
################
7-
8-
# Git Hub integration and bot rules
9-
/.github/ @jsquire @jhendrixMSFT @rickwinter @ronniegeraghty
10-
114
#############
125
# SDK (track2)
136
#############
147

158
# Catch all for non-code project files and unowned files | folders
16-
* @rickwinter @jhendrixMSFT
9+
/** @rickwinter @jhendrixMSFT
1710
/sdk/ @rickwinter @chlowell @richardpark-msft
1811

1912
# Samples
@@ -41,13 +34,10 @@
4134
/sdk/keyvault/ @chlowell @jhendrixMSFT
4235

4336
# PRLabel: %Service Bus
44-
/sdk/messaging/azservicebus @richardpark-msft @jhendrixMSFT
37+
/sdk/messaging/azservicebus/ @richardpark-msft @jhendrixMSFT
4538

4639
# PRLabel: %Event Hubs
47-
/sdk/messaging/azeventhubs @richardpark-msft @jhendrixMSFT
48-
49-
# PRLabel: %Service Bus
50-
/sdk/messaging/internal @richardpark-msft @jhendrixMSFT
40+
/sdk/messaging/azeventhubs/ @richardpark-msft @jhendrixMSFT
5141

5242
# PRLabel: %Monitor
5343
/sdk/monitor/ @gracewilcox @chlowell @jhendrixMSFT
@@ -58,12 +48,17 @@
5848
# PRLabel: %Storage
5949
/sdk/storage/ @siminsavani-msft @souravgupta-msft @tasherif-msft @jhendrixMSFT @gapra-msft
6050

51+
################
52+
# Automation
53+
################
54+
55+
# Git Hub integration and bot rules
56+
/.github/ @benbp @jsquire @jhendrixMSFT @rickwinter @ronniegeraghty
57+
6158
###########
6259
# Eng Sys
6360
###########
6461
/eng/ @benbp @weshaggard
65-
/**/ci.yml @benbp
66-
/**/tests.yml @benbp
6762

6863
# PRLabel: %EngSys
6964
/sdk/template/ @benbp @weshaggard

0 commit comments

Comments
 (0)