Skip to content

[Compute] Fixing RunCommand API and adding missing ScaleSet Properties#4549

Merged
dsgouda merged 1 commit intoAzure:psSdkJson6from
hyonholee:runcommand_fix
Jul 19, 2018
Merged

[Compute] Fixing RunCommand API and adding missing ScaleSet Properties#4549
dsgouda merged 1 commit intoAzure:psSdkJson6from
hyonholee:runcommand_fix

Conversation

@hyonholee
Copy link
Contributor

@hyonholee hyonholee commented Jul 16, 2018

I didn't update the version because the existing version (21.0.0) is not yet published to nuget.

Azure/azure-rest-api-specs#3364

Azure/azure-rest-api-specs#3406

Description


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@hyonholee
Copy link
Contributor Author

I didn't update the version because the existing version (21.0.0) is not yet published to nuget.

@hyonholee
Copy link
Contributor Author

The build failure doesn't seem to be related with this PR. Could you check why the build fails? Thanks.

@hyonholee
Copy link
Contributor Author

Please review this PR. I need this to fix RunCommand issue for Powershell.

@hyonholee hyonholee assigned hyonholee and dsgouda and unassigned hyonholee Jul 17, 2018
Copy link
Contributor

@dsgouda dsgouda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for the most part, please merge the 2 commits into a single one and we are good to merge

@hyonholee
Copy link
Contributor Author

@dsgouda now I have only one commit. Thanks.

@dsgouda
Copy link
Contributor

dsgouda commented Jul 18, 2018

@hyonholee you probably need to pull down the latest changes and update the PR again

@hyonholee
Copy link
Contributor Author

@dsgouda the failed test doesn't seem to be related with my PR. Can you retry it?

Copy link
Contributor

@dsgouda dsgouda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dsgouda dsgouda merged commit fb14ab0 into Azure:psSdkJson6 Jul 19, 2018
@hyonholee hyonholee deleted the runcommand_fix branch August 15, 2018 00:44
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.

2 participants