Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Adding new allowed failure script for Az bot #1269

Merged
merged 2 commits into from
Jun 3, 2017

Conversation

vishrutshah
Copy link
Contributor

@vishrutshah vishrutshah commented May 31, 2017

Checking in the small script for the bot independent of the others. Keeping them in the "allowed failure" list for a while

@vishrutshah vishrutshah reopened this May 31, 2017
@vishrutshah vishrutshah reopened this May 31, 2017
@vishrutshah vishrutshah reopened this May 31, 2017
@vishrutshah vishrutshah changed the title [WIP][CI] Az bot linter analyzer [CI] Az bot linter analyzer May 31, 2017
@azuresdkci
Copy link
Contributor

Hi There,

I am the Moment of Truth Azure bot. I am here to help. My task to analyze the situation from the AutoRest linter perspective. Please review the below analysis result:

File: keyvault/2016-10-01/swagger/keyvault.json
Number of linter errors/warnings before the PR: 107
Number of linter errors/warnings after the PR: 107

File: monitor/2016-09-01/swagger/metrics_API.json
Number of linter errors/warnings before the PR: 1
Number of linter errors/warnings after the PR: 1

Thanks for your co-operation.

@vishrutshah
Copy link
Contributor Author

Merging in as the CI passes and changes is kept inside the "Allowed Failure"

@vishrutshah vishrutshah changed the title [CI] Az bot linter analyzer [CI] Adding new allowed failure script for Az bot Jun 3, 2017
@vishrutshah vishrutshah merged commit a7586a8 into Azure:master Jun 3, 2017
@AutorestCI
Copy link

No modification for Python

@AutorestCI
Copy link

No modification for Ruby

@AutorestCI
Copy link

No modification for NodeJS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants