Skip to content

Commit

Permalink
Merge branch 'test-scenario-main' into ruowan/test-scenario
Browse files Browse the repository at this point in the history
  • Loading branch information
ruowan authored Mar 19, 2021
2 parents e9d1158 + af6acce commit 52c92ac
Show file tree
Hide file tree
Showing 4,822 changed files with 632,289 additions and 10,056 deletions.
The diff you're trying to view is too large. We only load the first 3000 changed files.
2 changes: 1 addition & 1 deletion .github/CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
### *This is an outdated document. Please refer the new [wiki](https://github.com/Azure/adx-documentation-pr/wiki) for up to date details.*
### *This is an outdated document. Please refer the new [wiki](https://github.com/Azure/azure-rest-api-specs/wiki) for up to date details.*
([`Link your GitHub account`](https://repos.opensource.microsoft.com/) to the 'Azure' organization for access.)

8 changes: 4 additions & 4 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ Please ensure to add changelog with this PR by answering the following questions
- [ ] No, no need to refresh for updates in this PR

### Contribution checklist:
- [ ] I commit to follow the [Breaking Change Policy](http://aka.ms/bcforapi) of no breaking changes
- [ ] I commit to follow the [Breaking Change Policy](http://aka.ms/bcforapi) of "no breaking changes"
- [ ] I have reviewed the [documentation](https://aka.ms/ameonboard) for the workflow.
- [ ] [Validation tools](https://aka.ms/swaggertools) were run on swagger spec(s) and errors have all been fixed in this PR. [How to fix?](https://aka.ms/ci-fix)

Expand All @@ -40,13 +40,13 @@ If any further question about AME onboarding or validation tools, please view th
- [ ] 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
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the [Breaking Change Policy](http://aka.ms/bcforapi).
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the [Breaking Change Policy](http://aka.ms/bcforapi).

- [ ] Removing API(s) in stable version
- [ ] Removing properties in stable version
- [ ] Removing API version(s) in stable version
- [ ] Updating API in stable version with Breaking Change Validation errors
- [ ] Updating API(s) in preview over 1 year
- [ ] Updating API in stable or public preview version with Breaking Change Validation errors
- [ ] Updating API(s) in public preview over 1 year (refer to [Retirement of Previews](https://dev.azure.com/msazure/AzureWiki/_wiki/wikis/AzureWiki.wiki/37683/Retirement-of-Previews))

**Action**: to initiate an evaluation of the breaking change, create a new intake using the [template for breaking changes](https://aka.ms/Breakingchangetemplate). Addition details on the process and office hours are on the [Breaking change Wiki](https://dev.azure.com/msazure/AzureWiki/_wiki/wikis/AzureWiki.wiki/37684/Breaking-Changes).

Expand Down
7 changes: 7 additions & 0 deletions .github/comment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,4 +50,11 @@
- "I have reviewed the [documentation](https://aka.ms/ameonboard) for the workflow."
onOpenedComments: "Hi, @${PRAuthor} Thanks for your PR. I am workflow bot for review process. Here are some small tips. </br> <li> Please ensure to do self-check against checklists in first PR comment.</li> <li> PR assignee is the person auto-assigned and responsible for your current PR reviewing and merging. </li> <li> For specs comparison cross API versions, Use [API Specs Comparison Report Generator](${openapiHub}/tools/diff?pr=${owner}/${repo}/${PRNumber})</li> <li> If there is CI failure(s), to fix CI error(s) is mandatory for PR merging; or you need to provide justification in PR comment for explanation. [How to fix?](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md)</li></br> Any feedback about review process or workflow bot, pls contact swagger and tools team. vsswagger@microsoft.com"

- rule:
type: checkbox
keywords:
- "update existing version to fix swagger quality issue in s360"
onCheckedLabels:
- FixS360
onCheckedComments: "In the Window Period to fix mismatches between swagger and service, when PR is labelled with “FixS360”, breaking change can be approved by PR assignee; the Azure Breaking Change Board is no longer required to approve the PR. Please ensure to clarify what s360 action items to be solved, and @ mention PR assignee for awareness. Please check this wiki [Window to Fix Broken](https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/405/Window-to-Fix-Broken) for more details."

20 changes: 7 additions & 13 deletions .github/pull_request_assignment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,12 @@
paths: "specification/@(resources|powerplatform|appplatform)/**"
reviewers:
- raych1


- rule:
paths: "specification/@(security|edgeorder|synapse)/**"
reviewers:
- ruowan

- rule:
paths: "specification/sql/**"
reviewers:
Expand Down Expand Up @@ -78,7 +83,6 @@
- "specification/authorization/**"
reviewers:
- qianwens
- yungezz

- rule:
paths:
Expand Down Expand Up @@ -106,7 +110,7 @@
- allenjzhang
- rule:
paths:
- "specification/@(databricks|containerregistry)/**"
- "specification/containerregistry/**"
reviewers:
- fengzhou-msft

Expand All @@ -115,7 +119,6 @@
- "specification/@(containerservice|containerinstance)/**"
reviewers:
- PhoenixHe-msft
- yungezz

- rule:
paths:
Expand Down Expand Up @@ -162,12 +165,6 @@
- josefree
- akning-ms

- rule:
paths:
- "documentation/**"
reviewers:
- josefree
- akning-ms

- rule:
paths:
Expand All @@ -187,16 +184,13 @@
paths:
- "**"
reviewers:
- yungezz
- erich-wang
- PhoenixHe-msft
- lirenhe
- weidongxu-microsoft
- ArcturusZhang
- xccc-msft
- raych1
- ChenTanyi
- fengzhou-msft
- njuCZ
- akning-ms
- leni-msft
Expand Down
31 changes: 28 additions & 3 deletions .prettierrc.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,30 @@
{
"plugins": ["./scripts/prettier-swagger-plugin"],
"parser": "json-swagger",
"printWidth": 20
"overrides": [
{
"files": [
"specification/**/*.json",
"documentation/swagger-bootstrap.json",
"documentation/samplefiles/**/*.json"
],
"options": {
"plugins": [
"./scripts/prettier-swagger-plugin"
],
"parser": "json-swagger",
"printWidth": 20
}
},
{
"files": [
"**/*.json"
],
"options": {
"parser": "json",
"printWidth": 20
},
"excludeFiles": [
"specification/**/*.json"
]
}
]
}
7 changes: 7 additions & 0 deletions .vscode/extensions.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"recommendations": [
"EditorConfig.EditorConfig",
"esbenp.prettier-vscode",
"docsmsft.docs-authoring-pack"
]
}
27 changes: 27 additions & 0 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"type": "node",
"request": "launch",
"name": "Mocha Tests",
"program": "${workspaceFolder}/node_modules/mocha/bin/_mocha",
"args": [
"--timeout",
"999999",
"--colors",
"${workspaceFolder}/test/syntax.js"
],
"internalConsoleOptions": "openOnSessionStart"
},
{
"type": "node",
"request": "launch",
"name": "Launch Program",
"program": "${file}"
}
]
}
27 changes: 27 additions & 0 deletions .vscode/settings.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
// Place your settings in this file to overwrite default and user settings.
{
"json.schemas": [
{
"fileMatch": [
"/**/specification/*.json"
],
"url": "https://raw.githubusercontent.com/Azure/autorest/master/packages/libs/autorest-schemas/swagger-extensions.json"
},
{
"fileMatch": [
"/**/examples/*.json"
],
"url": "https://raw.githubusercontent.com/Azure/autorest/master/packages/libs/autorest-schemas/example-schema.json"
},
{
"fileMatch": [
"/**/composite*.json"
],
"url": "https://raw.githubusercontent.com/Azure/autorest/master/packages/libs/autorest-schemas/composite-swagger.json"
}
],
"typescript.tsdk": "node_modules\\typescript\\lib",
"[json]": {
"editor.defaultFormatter": "esbenp.prettier-vscode"
}
}
9 changes: 6 additions & 3 deletions CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
/specification/adp/ @Azure/adp
/specification/analysisservices/ @taiwu
/specification/alertsmanagement/ @ofirmanor @olalavi @erangon @orieldar @ilaizi @shakednai1 @khaboasb @orenhor
/specification/apimanagement/ @promoisha @solankisamir
Expand Down Expand Up @@ -49,7 +50,7 @@
/specification/operationalinsights/ @sw47
/specification/operationsmanagement/ @dashimi16
/specification/peering/ @rileymckenna
/specification/policyinsights/ @bulentelmaci
/specification/policyinsights/ @pilor
/specification/postgresql/ @qingqingyuan
/specification/powerbidedicated/ @tarostok
/specification/provisioningservices/ @kvish
Expand All @@ -60,7 +61,7 @@
/specification/relay/ @sethmanheim @v-ajnava
/specification/resources/ @Azure/arm-template-deployments @rajshah11 @vivsriaus
/specification/scheduler/ @pinwang81
/specification/search/data-plane/ @brjohnstmsft @arv100kri @bleroy @heaths @AlexGhiondea
/specification/search/data-plane/ @brjohnstmsft @arv100kri @bleroy @heaths @AlexGhiondea @Mohit-Chakraborty
/specification/search/resource-manager/ @abhi1509 @miwelsh @tjacobhi
/specification/serialconsole/ @amitchat @craigw @asinn826
/specification/service-map/ @daveirwin1
Expand All @@ -78,4 +79,6 @@
/specification/trafficmanager/ @allencal @hrkulkarmsft
/specification/web/ @naveedaz @Azure/azure-app-service-control-plane
/profile/ @shahabhijeet
/specification/**/*.md @lirenhe
/specification/**/readme.typescript.md @qiaozha
/specification/**/readme.go.md @ArcturusZhang
/specification/**/readme.python.md @jsntcy @msyyc
42 changes: 40 additions & 2 deletions cSpell.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,9 @@
"description": "Project Words"
}
],
"flagWords": [],
"flagWords": [
"teh"
],
"ignorePaths": [
"node_modules/**",
"**/examples/**"
Expand Down Expand Up @@ -43,6 +45,24 @@
"Tera"
]
},
{
"filename": "**/specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json",
"words": [
"Tera"
]
},
{
"filename": "**/specification/databox/resource-manager/Microsoft.DataBox/stable/2020-04-01/databox.json",
"words": [
"Tera"
]
},
{
"filename": "**/specification/databox/resource-manager/Microsoft.DataBox/stable/2020-11-01/databox.json",
"words": [
"Tera"
]
},
{
"filename": "**/specification/servicebus/resource-manager/Microsoft.ServiceBus/stable/2017-04-01/servicebus.json",
"words": [
Expand Down Expand Up @@ -549,6 +569,24 @@
"Speciality"
]
},
{
"filename": "**/specification/cognitiveservices/data-plane/Face/stable/v1.0/Face.json",
"words": [
"addfacefromurl",
"deleteface",
"detectwithstream",
"detectwithurl",
"facelist",
"findsimilar",
"getface",
"largefacelist",
"largepersongroup",
"largepersongroupperson",
"persongroup",
"persongroupperson",
"verifyfacetoface"
]
},
{
"filename": "**/specification/storageimportexport/resource-manager/Microsoft.ImportExport/stable/2016-11-01/storageimportexport.json",
"words": [
Expand Down Expand Up @@ -764,4 +802,4 @@
]
}
]
}
}
Loading

0 comments on commit 52c92ac

Please sign in to comment.