Skip to content

Commit

Permalink
Update code owners file (#16657)
Browse files Browse the repository at this point in the history
* Moving the section to the end of the file as the order of processing rules is based on the file order.

* Temporary fix to remove xgithubtriage as an entry for the 'storage' folder.

The xgithubtriage team will be notified by incoming issues but should not be added as a PR reviewer.
  • Loading branch information
AlexGhiondea authored Nov 6, 2020
1 parent 96e4f11 commit 55fda6f
Showing 1 changed file with 19 additions and 21 deletions.
40 changes: 19 additions & 21 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -25,26 +25,6 @@
# Smoke tests
/common/SmokeTests/ @AlexGhiondea @schaabs @heaths @tg-msft @jsquire


# ######## Management Plane ########

/**/*Management*/ @allenjzhang @m-nash @markcowl @YalinLi0312 @bquantump @nisha-bhatia
/**/Azure.ResourceManager*/ @allenjzhang @m-nash @markcowl @YalinLi0312 @bquantump @nisha-bhatia

# Reviewers to double check any API changes
/sdk/**/api/ @KrzysztofCwalina @tg-msft

# Update AutoRest C#
/eng/CodeGeneration.targets @pakrym @ShivangiReja


# ######## Eng Sys ########
/eng/ @weshaggard @chidozieononiwu @mitchdenny @danieljurek
/eng/mgmt/ @allenjzhang @m-nash @markcowl
/**/tests.yml @danieljurek
/**/ci.yml @mitchdenny


# ######## Services ########

# ServiceLabel: %AAD %Service Attention
Expand Down Expand Up @@ -560,7 +540,7 @@ sdk/trafficmanager/Microsoft.Azure.Management.TrafficManager/ @tmsupp
/sdk/storage/Microsoft.Azure.WebJobs.*/ @amishra-dev @seanmcc-msft @amnguye @kasobol-msft @tg-msft @fabiocav @mathewc

# ServiceLabel: %Storage %Service Attention
/sdk/storage/ @xgithubtriage
#/<NotInRepo>/ @xgithubtriage

# ServiceLabel: %Storsimple %Service Attention
/sdk/storsimple/Microsoft.Azure.Management.StorSimple/ @anoobbacker @ganzee @manuaery @patelkunal
Expand Down Expand Up @@ -594,3 +574,21 @@ sdk/trafficmanager/Microsoft.Azure.Management.TrafficManager/ @tmsupp

# ServiceLabel: %Web Apps %Service Attention
#/<NotInRepo>/ @AzureAppServiceCLI @antcp

# ######## Management Plane ########

/**/*Management*/ @allenjzhang @m-nash @markcowl @YalinLi0312 @bquantump @nisha-bhatia
/**/Azure.ResourceManager*/ @allenjzhang @m-nash @markcowl @YalinLi0312 @bquantump @nisha-bhatia

# Reviewers to double check any API changes
/sdk/**/api/ @KrzysztofCwalina @tg-msft

# Update AutoRest C#
/eng/CodeGeneration.targets @pakrym @ShivangiReja


# ######## Eng Sys ########
/eng/ @weshaggard @chidozieononiwu @mitchdenny @danieljurek
/eng/mgmt/ @allenjzhang @m-nash @markcowl
/**/tests.yml @danieljurek
/**/ci.yml @mitchdenny

0 comments on commit 55fda6f

Please sign in to comment.