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

A few misc cleanups #6324

Merged
merged 3 commits into from Aug 9, 2018

Conversation

Projects
None yet
2 participants
@illicitonion
Copy link
Contributor

illicitonion commented Aug 9, 2018

See each commit for details

@illicitonion illicitonion requested a review from stuhood Aug 9, 2018

@stuhood

stuhood approved these changes Aug 9, 2018

@illicitonion illicitonion merged commit 0f4b2c6 into pantsbuild:master Aug 9, 2018

1 check passed

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

@illicitonion illicitonion deleted the twitter:dwagnerhall/remotezinc/misc branch Aug 9, 2018

cosmicexplorer added a commit to cosmicexplorer/pants that referenced this pull request Aug 10, 2018

A few misc cleanups (pantsbuild#6324)
* get_buildroot() always returns a string

* Simplify expression with try_future macro

* Don't truncate fingerprints in repr

This makes it much easier to look things up in fs_util

CMLivingston pushed a commit to CMLivingston/pants that referenced this pull request Aug 27, 2018

A few misc cleanups (pantsbuild#6324)
* get_buildroot() always returns a string

* Simplify expression with try_future macro

* Don't truncate fingerprints in repr

This makes it much easier to look things up in fs_util
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment