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

Update Vulnerability Assessment swaggers #4394

Merged
merged 1 commit into from
Nov 6, 2018
Merged

Update Vulnerability Assessment swaggers #4394

merged 1 commit into from
Nov 6, 2018

Conversation

yaakoviyun
Copy link
Member

@yaakoviyun yaakoviyun commented Nov 4, 2018

Update Vulnerability Assessment swaggers

Latest improvements:

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

  • I have reviewed the documentation for the workflow.
  • Validation tools were run on swagger spec(s) and have all been fixed in this PR.
  • The OpenAPI Hub was used for checking validation status and next steps.

Update Vulnerability Assessment swaggers
@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-ruby

Nothing to generate for azure-sdk-for-ruby

@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#3680

@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-js

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-js#420

@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-java

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-java#2138

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-go

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-go#3248

@AutorestCI
Copy link

AutorestCI commented Nov 4, 2018

Automation for azure-sdk-for-node

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-node#4039

@yaakoviyun
Copy link
Member Author

@annatisch - can you please review and merge?

@annatisch
Copy link
Member

Thanks @yaakoviyun - from what I can see in the diff, the only changes in this PR are:

  • The list operation has been moved/reordered, but remains unchanged.
  • Removed a duplicate redundant model definition VulnerabilityAssessmentRecurringScansProperties
  • Made some updates to error descriptions

Is that correct?

@yaakoviyun
Copy link
Member Author

yaakoviyun commented Nov 6, 2018 via email

@annatisch annatisch merged commit 1a5061c into Azure:master Nov 6, 2018
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.

4 participants