Skip to content

[Ready for review] Regenerated the Batch management apis as swagger is updated.#1089

Merged
anuchandy merged 4 commits into
Azure:masterfrom
anudeepsharma:BatchV2
Sep 20, 2016
Merged

[Ready for review] Regenerated the Batch management apis as swagger is updated.#1089
anuchandy merged 4 commits into
Azure:masterfrom
anudeepsharma:BatchV2

Conversation

@anudeepsharma

@anudeepsharma anudeepsharma commented Sep 20, 2016

Copy link
Copy Markdown
Contributor

This change is just generation with new swagger, plus resolved errors accordingly.

@anudeepsharma anudeepsharma force-pushed the BatchV2 branch 2 times, most recently from 8143107 to 9978fe0 Compare September 20, 2016 02:17
@azuresdkci

Copy link
Copy Markdown
Contributor

Runtime changes detected. pull request created. CI running: Build Status

@anuchandy

Copy link
Copy Markdown
Member

lgtm, can be merged once CI passes.

@martinsawicki

Copy link
Copy Markdown

CI fails with this:

4161[ERROR] /home/travis/build/Azure/azure-sdk-for-java/azure-mgmt-batch/src/main/java/com/microsoft/azure/management/batch/implementation/BatchAccountImpl.java:73:64: Name 'BatchAccountInner' must match pattern '^[a-z][a-zA-Z0-9]*$'. [ParameterName]

Huh?

@martinsawicki

Copy link
Copy Markdown

sounds like a parameter name starts with a capitalized letter instead of small?

@anuchandy anuchandy merged commit 55e98a5 into Azure:master Sep 20, 2016
@anudeepsharma anudeepsharma deleted the BatchV2 branch September 23, 2016 20:01
jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Feb 27, 2017
Fixing a breaking change introduced due to RestClient, manually fixin…
jianghaolu pushed a commit to jianghaolu/azure-sdk-for-java that referenced this pull request Feb 27, 2017
[Ready for review] Regenerated the Batch management apis as swagger is updated.
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