Skip to content

Added customization in RecoveryServices client and updated Microsoft.Rest.ClientRuntime to 2.3.10#3657

Merged
shahabhijeet merged 3 commits intoAzure:psSdkJson6from
sonathan:psSdkJson6
Sep 7, 2017
Merged

Added customization in RecoveryServices client and updated Microsoft.Rest.ClientRuntime to 2.3.10#3657
shahabhijeet merged 3 commits intoAzure:psSdkJson6from
sonathan:psSdkJson6

Conversation

@sonathan
Copy link
Copy Markdown
Contributor

@sonathan sonathan commented Sep 5, 2017

Following items in this PR:

  1. Added customizations in RecoveryServices Client.
  2. Updated Microsoft.Rest.ClientRuntime to use 2.3.10

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.

@msftclas
Copy link
Copy Markdown

msftclas commented Sep 5, 2017

@sonathan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

<TargetFrameworks>net452;netstandard1.4</TargetFrameworks>
</PropertyGroup>
<ItemGroup>
<PackageReference Update="Microsoft.Rest.ClientRuntime" Version="2.3.10" />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@sonathan this is not needed.
We control the common packages when we have tested and ready to move to a new version of clientruntime throughout the repo.
So when the new version is out and ready to be consumed, this change will be done for the entire repo.
So please revert this change.
If we allow this change, controlling ClientRuntime version across repo will be fragmented and not manageable.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@shahabhijeet Thank-you for letting me know of this. I'll update this in next commit.

DisableDispose = disableDispose;
}

public void SetHttpClient(HttpClient client)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FYI
@sonathan the new version 2.3.9 has an overload that will allow you to control disposing of the provided client

@shahabhijeet shahabhijeet merged commit 15ed7d0 into Azure:psSdkJson6 Sep 7, 2017
JasonYang-MSFT pushed a commit to JasonYang-MSFT/azure-sdk-for-net that referenced this pull request Dec 8, 2017
…Rest.ClientRuntime to 2.3.10 (Azure#3657)

* Added customization on RecoveryServices client and updated Microsoft.Rest.ClientRuntime to 2.3.10

* Reverting Microsoft.Rest.ClientRuntime to 2.3.8

* Removing ClientRuntime Version specification
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.

3 participants