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

Fix the wheel count check for pex building #7933

Merged
merged 1 commit into from Jun 22, 2019

Conversation

Projects
None yet
2 participants
@stuhood
Copy link
Member

commented Jun 22, 2019

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.

@stuhood stuhood requested a review from Eric-Arellano Jun 22, 2019

@Eric-Arellano
Copy link
Contributor

left a comment

Good catch. Thanks Stu!

Nit: please rename "whl" to "wheel" in PR title

@stuhood stuhood changed the title Fix the whl count check for pex building Fix the wheel count check for pex building Jun 22, 2019

@stuhood stuhood merged commit 9bed8ae into pantsbuild:master Jun 22, 2019

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.