Skip to content

Conversation

@juniwang
Copy link
Member

Add new swagger specification and examples for azure SingalR service Resource provider. The swagger file is generated using Swashbuckle and is swagger-2.0 based.

Tests performed:

  • autoRest <path to readme.md> --azure-validator=true passed with two warnings. Both warnings suggests replacing boolean property with enum however they must to be boolean according to ARM api spec.
  • oav validate-spec <spec-path> passed
  • oav validate-example <spec-path> passed

A PR to the private repo has been reviewed and approved.

@sergey-shandar sergey-shandar self-assigned this Apr 17, 2018
@AutorestCI
Copy link

AutorestCI commented Apr 17, 2018

Automation for azure-sdk-for-node

Nothing to generate for azure-sdk-for-node

@sergey-shandar
Copy link
Contributor

@AutorestCI
Copy link

AutorestCI commented Apr 17, 2018

Automation for azure-sdk-for-python

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

@AutorestCI
Copy link

AutorestCI commented Apr 17, 2018

Automation for azure-libraries-for-java

The initial PR has been merged into your service PR:
AutorestCI/azure-libraries-for-java#84

@AutorestCI
Copy link

AutorestCI commented Apr 17, 2018

Automation for azure-sdk-for-go

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

@azuresdkciprbot
Copy link

Hi There,

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

💡 Please review potentially introduced Error(s)/Warning(s): Analysis Report 💡

File: specification/signalr/resource-manager/readme.md
Before the PR: Warning(s): 0 Error(s): 0
After the PR: Warning(s): 1 Error(s): 0

AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback

Thanks for your co-operation.

@sergey-shandar sergey-shandar merged commit b6cb0b6 into Azure:master Apr 17, 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.

5 participants