I'm not sure how the `:API: public` protocol is playing out in practice. I think publicizing this method was just an honest mistake in https://rbcommons.com/s/twitter/r/3655/.
De-publicize a FAPP private method.
Review Request #3750 — Created April 22, 2016 and submitted
|molsen, nhoward_tw, zundel|
This also fixes some >100 cols formatting. src/python/pants/java/distribution/distribution.py | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-)
Locally has the same set of failures before and after:
./pants test tests/python/pants_test/java/distribution
CI went green here:
Status: Closed (submitted)
Now on master: git log -1 origin/master commit b2a4960730dcf4ba256e679af9f782dcc914832d Author: John Sirois <firstname.lastname@example.org> Date: Sun Apr 24 10:09:33 2016 -0600 De-publicize a FAPP private method. This also fixes some >100 cols formatting. Testing Done: Locally has the same set of failures before and after: `./pants test tests/python/pants_test/java/distribution` CI went green here: https://travis-ci.org/pantsbuild/pants/builds/125125707 Bugs closed: 2894, 3260 Reviewed at https://rbcommons.com/s/twitter/r/3750/