Skip to content

fix #69 by rolling back @azure/ms-rest-js to ^1.8.13#70

Merged
amarzavery merged 2 commits intoAzure:masterfrom
amarzavery:rollback
Aug 2, 2019
Merged

fix #69 by rolling back @azure/ms-rest-js to ^1.8.13#70
amarzavery merged 2 commits intoAzure:masterfrom
amarzavery:rollback

Conversation

@amarzavery
Copy link
Contributor

@amarzavery amarzavery commented Aug 2, 2019

@amarzavery amarzavery requested a review from a user August 2, 2019 19:03
@amarzavery amarzavery merged commit f0d65b3 into Azure:master Aug 2, 2019
@@ -1,4 +1,7 @@
# Changelog
## 2.0.4 - 2019/08/02
- Rolling back the min version of dependency `@azure/ms-rest-js` from `^2.0.3` to `^1.8.13` thereby fixing [#69](https://github.com/Azure/ms-rest-nodeauth/issues/69).
Copy link
Contributor

Choose a reason for hiding this comment

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

What about azure/ms-rest-azure-env? The breaking changes there from major version 1 to 2 doesn't affect anybody?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That shouldn't affect anyone. It is a good breaking change. We removed a dot from the storage suffix which shouldn't have been there in the first place. This change was made after a customer filed an issue. Since it was a breaking change we updated the major version of the package. However, it is a good change and we don't expect it to cause issues to our customers since the previous code was not usable.

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.

Breaking change in a patch version

3 participants