Remove stale example from 3rdparty_jvm.md #4112

Merged
merged 1 commit into from Dec 2, 2016

Conversation

Projects
None yet
3 participants
@stuhood
Member

stuhood commented Dec 2, 2016

Problem

The 3rdparty_jvm docs show a very outdated and ill-advised approach to fixing dependency issues.

Solution

Remove it, and improve formatting.

@peiyuwang

This comment has been minimized.

Show comment
Hide comment
@peiyuwang

peiyuwang Dec 2, 2016

Contributor

should we include another example for this use case? the exclude example below is relevant but does not directly address the use case.

Contributor

peiyuwang commented Dec 2, 2016

should we include another example for this use case? the exclude example below is relevant but does not directly address the use case.

@stuhood stuhood closed this Dec 2, 2016

@stuhood stuhood deleted the stuhood-patch-1 branch Dec 2, 2016

@stuhood stuhood restored the stuhood-patch-1 branch Dec 2, 2016

@stuhood stuhood reopened this Dec 2, 2016

@stuhood stuhood merged commit 3af3144 into master Dec 2, 2016

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@stuhood

This comment has been minimized.

Show comment
Hide comment
@stuhood

stuhood Dec 2, 2016

Member

(note to self: don't use the github "edit inline" support on pantsbuild/pants, as it creates a branch in that repo, which double-triggers travis... rather, use it in a fork)

Member

stuhood commented Dec 2, 2016

(note to self: don't use the github "edit inline" support on pantsbuild/pants, as it creates a branch in that repo, which double-triggers travis... rather, use it in a fork)

@stuhood stuhood deleted the stuhood-patch-1 branch Dec 2, 2016

lenucksi added a commit to lenucksi/pants that referenced this pull request Apr 25, 2017

Remove stale example from 3rdparty_jvm.md (#4112)
### Problem

The 3rdparty_jvm docs show a very outdated and ill-advised approach to fixing dependency issues.

### Solution

Remove it, and improve formatting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment