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
No longer release a Python 2 wheel or PEX #7888
No longer release a Python 2 wheel or PEX #7888
Conversation
Not to merge until Twitter has landed their Python 3 patch, which will hopefully be today or tomorrow. Otherwise all ready for review :) |
For some reason, `check_pants_pex_abi.py` cannot find `./pants.pex`. Maybe, just maybe, this is related..?
* Missing urllib imports * Not redirecting stdout to stderr
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.
nice!
There's no more need to parametrize docker_image_name. Removing that flexibility makes the code more explicit and easier to understand.
# Delete this line to force a full CI run for documentation-only changes. SKIP_FULL_CI # Documentation-only change.
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.
still lgtm!
### Problem #7888 fixed most of the locations that were expecting particular whl counts, but missed one. ### Solution Fix the last whl count check. ### Result Confirmed that `build-support/bin/release.sh -p` works.
We will now require Python 3 to run Pants for new releases (1.18.x or greater), per https://groups.google.com/forum/#!topic/pants-devel/-bMRstFtMZU.
Here, we remove Python 2 from all release related code, including:
-2
and--py2
logic fromrelease.sh
andpackages.py
.