Skip to content

Commit 8b0b1d3

Browse files
authored
[CODEOWNERS] - Make primary package owner first (Azure#20155)
### Packages impacted by this PR None ### Issues associated with this PR Resolves Azure#18210 ### Describe the problem that is addressed by this PR Today, vendors create issues and assign them based on a list that I provided them. The problem is that it can easily get out of sync as nobody would know to update it. I am working towards making CODEOWNERS the single source of truth for package ownership. It needs to start with the primary owner listed first, so that the vendors know who to assign it to. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? I am going to be honest, this effort might not go anywhere. But it's not a harmful thing to do. Besides, there's a bit of a chicken-and-egg problem right now. CODEOWNER list ordering is not used by the vendors because it's not ordered, but nobody will want to order it because the order has no impact. Between this and the efforts engsys is putting in place to parse this file I am hopeful we can get to a place where vendors can look at data derived from CODEOWNERS.
1 parent 5de8993 commit 8b0b1d3

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

.github/CODEOWNERS

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,9 @@
3636
# PRLabel: %Communication
3737
/sdk/communication/communication-identity/ @Azure/acs-identity-sdk
3838
/sdk/communication/communication-chat/ @LuChen-Microsoft
39-
/sdk/communication/communication-phone-numbers/ @miguhern @danielav7 @whisper6284 @AlonsoMondal
39+
/sdk/communication/communication-phone-numbers/ @whisper6284 @miguhern @danielav7 @AlonsoMondal
4040
/sdk/communication/communication-network-traversal/ @AriZavala2
41-
/sdk/communication/communication-sms/ @RoyHerrod @arifibrahim4
41+
/sdk/communication/communication-sms/ @arifibrahim4 @RoyHerrod
4242
/sdk/communication/communication-short-codes/ @danielav7 @AlonsoMondal @ericasp16
4343
/sdk/communication/communication-common/ @Azure/acs-identity-sdk
4444

@@ -58,10 +58,10 @@
5858
/sdk/digitaltwins/ @johngallardo @YoDaMa @olivakar
5959

6060
# PRLabel: %Event Grid
61-
/sdk/eventgrid/ @xirzec @ellismg
61+
/sdk/eventgrid/ @ellismg @xirzec
6262

6363
# PRLabel: %Event Hubs
64-
/sdk/eventhub/ @deyaaeldeen @jeremymeng @HarshaNalluru @ramya-rao-a
64+
/sdk/eventhub/ @jeremymeng @deyaaeldeen @HarshaNalluru @ramya-rao-a
6565

6666
# PRLabel: %Azure.Identity
6767
/sdk/identity/ @sadasant @witemple-msft @schaabs
@@ -76,10 +76,10 @@
7676
/sdk/instrumentation/ @maorleger @joheredi
7777

7878
# PRLabel: %Service Bus
79-
/sdk/servicebus/ @deyaaeldeen @jeremymeng @HarshaNalluru @ramya-rao-a @richardpark-msft
79+
/sdk/servicebus/ @jeremymeng @deyaaeldeen @HarshaNalluru @ramya-rao-a
8080

8181
# PRLabel: %Storage
82-
/sdk/storage/ @XiaoningLiu @jeremymeng @vinjiang @jiacfan @ljian3377 @EmmaZhu
82+
/sdk/storage/ @EmmaZhu @XiaoningLiu @jeremymeng @vinjiang @jiacfan @ljian3377
8383

8484
# PRLabel: %Synapse
8585
/sdk/synapse/ @joheredi
@@ -100,7 +100,7 @@
100100
/sdk/documenttranslator/ @joheredi
101101

102102
# PRLabel: %WebPubSub
103-
/sdk/web-pubsub/ @xirzec @bterlson @vicancy
103+
/sdk/web-pubsub/ @vicancy @xirzec @bterlson
104104

105105
# PRLabel: %EngSys
106106
/sdk/template/ @praveenkuttappan @mikeharder @weshaggard @benbp
@@ -114,16 +114,16 @@
114114
/sdk/schemaregistry/ @deyaaeldeen @nguerrera
115115

116116
# PRLabel: %Cognitive - Form Recognizer
117-
/sdk/formrecognizer/ @jeremymeng @witemple-msft
117+
/sdk/formrecognizer/ @witemple-msft @jeremymeng
118118

119119
# PRLabel: %Cognitive - Metrics Advisor
120-
/sdk/metricsadvisor/ @jeremymeng @KarishmaGhiya
120+
/sdk/metricsadvisor/ @KarishmaGhiya @jeremymeng
121121

122122
# PRLabel: %Cognitive - Anomaly Detector
123123
/sdk/anomalydetector/ @conhua @mengaims @juaduan @moreOver0
124124

125125
# PRLabel: %Search
126-
/sdk/search/ @xirzec @sarangan12
126+
/sdk/search/ @sarangan12 @xirzec
127127

128128
/sdk/applicationinsights/applicationinsights-query/ @divyajay @alongafni
129129
/sdk/operationalinsights/loganalytics/ @divyajay @alongafni

0 commit comments

Comments
 (0)