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

Add exports to several request policy factory methods #336

Merged
merged 2 commits into from
Mar 19, 2019

Conversation

jeremymeng
Copy link
Member

They are useful when customizing the requestPolicyFactories array of a
ServiceClientOptions instance.

Resolves #334.

They are useful when customizing the requestPolicyFactories array of a
ServiceClientOptions instance.
@jeremymeng jeremymeng requested a review from a user March 18, 2019 20:36
@ghost ghost added the in progress label Mar 18, 2019
@kpajdzik
Copy link
Contributor

LGTM. @daschult, what do you think?

@kpajdzik
Copy link
Contributor

@jeremymeng, do you mind updating the version in package.json to 1.8.0? I can't push to your fork. After that it's good to go.

@jeremymeng
Copy link
Member Author

Version updated. Thank all for reviewing!

@kpajdzik
Copy link
Contributor

@jeremymeng I forgot about version from Constants.ts (https://github.com/Azure/ms-rest-js/blob/master/lib/util/constants.ts#L11). If you could update that as well, I'd appreciate a lot :) Sorry for missing that!

@jeremymeng jeremymeng force-pushed the export-request-policy-factories branch from 36422bf to e6805d0 Compare March 19, 2019 18:03
@jeremymeng
Copy link
Member Author

@kpajdzik no problem!

@kpajdzik kpajdzik merged commit 8c54b39 into Azure:master Mar 19, 2019
@kpajdzik
Copy link
Contributor

@jeremymeng, This should be published in the next hour. Thanks for your contribution :)

@jeremymeng jeremymeng deleted the export-request-policy-factories branch March 19, 2019 18:33
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