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

missing-deps-suggest outputs buildozer commands if path to buildozer is set #5638

Merged
merged 1 commit into from Mar 29, 2018

Conversation

Projects
None yet
5 participants
@illicitonion
Copy link
Contributor

illicitonion commented Mar 29, 2018

No description provided.

@dotordogh
Copy link
Contributor

dotordogh left a comment

Awesome!! Thank you so much!!

@stuhood

This comment has been minimized.

Copy link
Member

stuhood commented Mar 29, 2018

Please wait for a review from Yi: pants has a buildozer subsystem, and he might suggest linking to that (in the future...?).

@wisechengyi
Copy link
Contributor

wisechengyi left a comment

Lgtm. We can revisit this once the subsystem is ready.

@stuhood stuhood merged commit d7478a2 into pantsbuild:master Mar 29, 2018

1 check passed

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

@stuhood stuhood deleted the twitter:dwagnerhall/missingdepssuggest/buildozer branch Mar 29, 2018

@baroquebobcat

This comment has been minimized.

Copy link
Contributor

baroquebobcat commented Mar 29, 2018

Sweet!

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