Fix issue with detecting existing comment for tryit url (#458)

This commit is contained in:
Timothee Guerin 2022-04-19 09:29:35 -07:00 коммит произвёл GitHub
Родитель fea04034f2
Коммит 94cdeb9818
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
3 изменённых файлов: 18 добавлений и 4 удалений

Просмотреть файл

@ -221,3 +221,14 @@ the command line following the steps above, or build its Release
configuration in Visual Studio, then you can install it by configuration in Visual Studio, then you can install it by
double-clicking on packages/cadl-vs/Microsoft.Cadl.VisualStudio.vsix double-clicking on packages/cadl-vs/Microsoft.Cadl.VisualStudio.vsix
that gets produced. that gets produced.
# Pull request
## Trigger Cadl Playground Try It build
For contributors of the repo the build will trigger automatically but for other's forks it will need a manual trigger from a contributor.
As a contributor you can run the following command to trigger the build and create a cadl playground link for this PR.
```
/azp run Cadl Pull Request Try It
```

Просмотреть файл

@ -37,5 +37,5 @@ jobs:
inputs: inputs:
gitHubConnection: "internal (1)" gitHubConnection: "internal (1)"
repositoryName: "$(Build.Repository.Name)" repositoryName: "$(Build.Repository.Name)"
comment: "You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/$(System.PullRequest.PullRequestNumber)/" comment: "<!-- _CADL_TRYIT_COMMENT_ -->\nYou can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/$(System.PullRequest.PullRequestNumber)/"
condition: and(succeeded(), ne(variables['SKIP_COMMENT'], 'true')) condition: and(succeeded(), ne(variables['SKIP_COMMENT'], 'true'))

Просмотреть файл

@ -17,8 +17,11 @@ async function main() {
const result = await request("GET", url); const result = await request("GET", url);
const data = JSON.parse(result); const data = JSON.parse(result);
const azoComments = data.filter((x) => x.user?.login === AZP_USERID); const azoComments = data.filter((x) => x.user?.login === AZP_USERID);
console.log(`Found ${azoComments.length} comments from Azure Pipelines.`); console.log(`Found ${azoComments.length} comment(s) from Azure Pipelines.`);
if (azoComments.length > 0) {
const tryItComments = data.filter((x) => x.body.includes("_CADL_TRYIT_COMMENT_"));
console.log(`Found ${azoComments.length} Cadl Try It comment(s)`);
if (tryItComments.length > 0) {
console.log("##vso[task.setvariable variable=SKIP_COMMENT;]true"); console.log("##vso[task.setvariable variable=SKIP_COMMENT;]true");
} }
} }
@ -37,7 +40,7 @@ async function request(method, url, postData) {
"User-Agent": "nodejs", "User-Agent": "nodejs",
}, },
}; };
console.log("P", params); console.log("Params", params);
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
const req = lib.request(params, (res) => { const req = lib.request(params, (res) => {