Merge branch 'master' of https://github.com/Azure/azure-rest-api-specs into master

Author: kangsun-ctrl

.azure-pipelines/PrettierCheck.yml

@@ -1,6 +1,5 @@
 jobs:
 - job: "PrettierCheck"
-  condition: "not(variables['PRIVATE'])"
   variables:
     NODE_OPTIONS: '--max-old-space-size=8192'
   pool:

.azure-pipelines/Spellcheck.yml

@@ -1,6 +1,5 @@
 jobs:
 - job: "Spellcheck"
-  condition: "not(variables['PRIVATE'])"
   variables:
     NODE_OPTIONS: '--max-old-space-size=8192'
   pool:

.github/PULL_REQUEST_TEMPLATE.md

@@ -14,6 +14,8 @@ If any further question about AME onboarding or validation tools, please view th
   - Adding a new API version
   - Adding a new service
 
+- [ ] Please ensure you've reviewed following [guidelines](https://aka.ms/rpguidelines) including [ARM resource provider contract](https://github.com/Azure/azure-resource-manager-rpc) and [REST guidelines](https://github.com/microsoft/api-guidelines/blob/vNext/azure/Guidelines.md). Estimated time (4 hours). This is required before you can request review from ARM API Review board.
+
 - [ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (*RP Manifest Approvers* team under <ins>Azure Resource Manager service</ins>) from IcM and reach out to them. 
 
 ### Breaking Change Review Checklist 

.github/pull_request_assignment.yml

@@ -16,10 +16,127 @@
       - lmazuel
       - tjprescott
 
+
+- rule:
+    paths:
+      - "specification/network/**"
+    reviewers:
+      - MyronFanQiu
+      - lirenhe
+
+- rule:
+    paths:
+      - "specification/@(applicationinsights|operationalinsights)/**"
+    reviewers:
+      - MyronFanQiu
+
+- rule:
+    paths: "specification/@(resources|powerplatform)/**"
+    reviewers:
+      - raych1
+      
+- rule:
+    paths: "specification/compute/**"
+    reviewers:
+      - ArcturusZhang
+      - fengzhou-msft
+      - yungezz
+
+- rule:
+    paths:
+      - "specification/machinelearningservices/**"
+    reviewers:
+      - ArcturusZhang
+
+- rule:
+    paths:
+      - "specification/@(azureactivedirectory|appplatform)/**"
+    reviewers:
+      - njuCZ
+
+- rule:
+    paths:
+      - "specification/authorization/**"
+    reviewers:
+      - qianwens
+      - yungezz
+
+- rule:
+    paths:
+      - "specification/datafactory/**"
+    reviewers:
+      - qianwens
+
+- rule:
+    paths:
+      - "specification/monitor/**"
+    reviewers:
+      - weidongxu-microsoft
+      - MyronFanQiu
+
+- rule:
+    paths:
+      - "specification/storage/**"
+    reviewers:
+      - qianwens
+      - Juliehzl
+
+
 - rule:	
     paths: "specification/mediaservices/**"	
     reviewers:	
       - allenjzhang
+- rule:
+    paths:
+      - "specification/@(keyvault|notificationhubs)/**"
+      - "specification/@(databricks|containerregistry)/**"
+    reviewers:
+      - fengzhou-msft
+
+- rule:
+    paths:
+      - "specification/@(containerservice|containerinstance)/**"
+    reviewers:
+      - PhoenixHe-msft
+      - fengzhou-msft
+      - yungezz
+
+- rule:
+    paths:
+      - "specification/mixedreality/**"
+      - "specification/hdinsight/**"
+    reviewers:
+      - leni-msft
+
+- rule:
+    paths:
+      - "specification/portal/**"
+    reviewers:
+      - PhoenixHe-msft
+
+- rule:
+    paths:
+      - "specification/web/**"
+    reviewers:
+      - weidongxu-microsoft
+
+- rule:
+    paths:
+      - "documentation/**"
+    reviewers:
+      - josefree
+      - akning-ms
+
+- rule:
+    paths:
+      - "**/package.json"
+      - ".github/**"
+      - "**/azure-pipelines.yml"
+      - ".azure-pipelines/**"
+      - ".azure-pipelines-preproduction/**"
+    reviewers:
+      - akning-ms
+      - PhoenixHe-msft
 
 - rule:	
     paths:	
@@ -45,6 +162,19 @@
     paths:
       - "**"
     reviewers:
-      - allenjzhang
-      - markcowl
-
+      - yungezz
+      - erich-wang
+      - PhoenixHe-msft
+      - lirenhe
+      - weidongxu-microsoft
+      - ArcturusZhang
+      - xccc-msft
+      - raych1
+      - ChenTanyi
+      - fengzhou-msft
+      - njuCZ
+      - akning-ms
+      - leni-msft
+      - qianwens
+      - ruowan
+      - jianyexi

CODEOWNERS

@@ -60,7 +60,7 @@
 /specification/relay/ @sethmanheim @v-ajnava
 /specification/resources/ @Tiano2017 @rajshah11 @vivsriaus
 /specification/scheduler/ @pinwang81
-/specification/search/data-plane/ @brjohnstmsft @arv100kri @bleroy
+/specification/search/data-plane/ @brjohnstmsft @arv100kri @bleroy @heaths @AlexGhiondea
 /specification/search/resource-manager/ @abhi1509 @miwelsh @tjacobhi
 /specification/serialconsole/ @amitchat @craigw @asinn826
 /specification/service-map/ @daveirwin1

custom-words.txt

@@ -282,6 +282,7 @@ Certificatethumbprint
 certverify
 Chademo
 chainer
+changedSince
 changepoint
 changestate
 CHECKACCESS