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

Fail build when setup-py run failed #6693

Merged
merged 4 commits into from Oct 29, 2018

Conversation

Projects
None yet
2 participants
@wisechengyi
Contributor

wisechengyi commented Oct 28, 2018

Fix for #6684 (pants reports success on setup-py failure)

wisechengyi added some commits Oct 28, 2018

@wisechengyi wisechengyi requested review from stuhood and jsirois Oct 28, 2018

@wisechengyi wisechengyi changed the title from Raise TaskError when setup-py run failed to Fail build when setup-py run failed Oct 28, 2018

wisechengyi added some commits Oct 28, 2018

fmt

@wisechengyi wisechengyi merged commit 021bde1 into pantsbuild:master Oct 29, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@wisechengyi wisechengyi deleted the wisechengyi:6684 branch Oct 29, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment