Skip to content

Conversation

karataliu
Copy link
Contributor

@karataliu karataliu commented Jan 6, 2021

Fixes: #134

version='a' has same semantic of '^a', which will auto pick version with same major version. This won't work for packages on alpha version, which can have breaking change.

@rylev
Copy link
Contributor

rylev commented Jan 6, 2021

Thanks! Since we wanted to update tokio anyway, I went ahead and fixed this in #136

@rylev rylev closed this Jan 6, 2021
@karataliu
Copy link
Contributor Author

@rylev The alpha.3 version still need to be locked, otherwise it could be broken in future?

@rylev
Copy link
Contributor

rylev commented Jan 7, 2021

@karataliu you're probably right since it seems oauth2 is not following any kind of semver guarantees in their alpha releases. I'm reopening this PR. Can you push a change to bump to alpha3?

@rylev rylev reopened this Jan 7, 2021
@karataliu
Copy link
Contributor Author

updated

Copy link
Contributor

@rylev rylev left a comment

Choose a reason for hiding this comment

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

Thanks!

@rylev rylev merged commit 30a3296 into Azure:master Jan 12, 2021
@karataliu karataliu deleted the fo2v branch January 12, 2021 14:10
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.

sdk/identity build broken due to oauth2 package update

2 participants