-
-
Notifications
You must be signed in to change notification settings - Fork 636
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
Resolve only stable releases by default. #4201
Conversation
NB: There is one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm and seems perfectly reasonable re: the one non---pre
-plumbed interpreter setup resolve.
Nifty, this fails the contrib shard since the contrib plugins depend on a pantsbuild.pants dev release for master builds - which is unstable, which is now disallowed by default. Fix diff coming... |
Turns out I had missed a fourth use of the pex resolve function - in the pants plugin resolver. This now plumbs |
5c6d5e2
to
35e6188
Compare
13496f7
to
eda170d
Compare
35e6188
to
3aaae84
Compare
Removed the |
Upgrade to pex `1.2.0` to pickup control over pre-release resolution and setup resolves to use only stable releases by default.
3aaae84
to
d80212c
Compare
Upgrade to pex `1.2.0` to pickup control over pre-release resolution and setup resolves to use only stable releases by default.
Upgrade to pex `1.2.0` to pickup control over pre-release resolution and setup resolves to use only stable releases by default.
Upgrade to pex
1.2.1
to pickup control over pre-release resolution andsetup resolves to use only stable releases by default.
Closes #4196