Un-publish the `jar` goal. #4095

Merged
merged 1 commit into from Nov 28, 2016

Conversation

Projects
None yet
2 participants
@jsirois
Member

jsirois commented Nov 23, 2016

This leaves the jar.create task installed and the jar goal
addressable from the command line, but no longer advertises the goal
since it's really not intended as-is for end users to invoke directly.

Un-publish the `jar` goal.
This leaves the `jar.create` task installed and the `jar` goal
addressable from the command line, but no longer advertises the goal
since it's really not intended as-is for end users to invoke directly.
@jsirois

This comment has been minimized.

Show comment
Hide comment
@jsirois

jsirois Nov 23, 2016

Member

This addresses the confusion caused by the jar goal being publicly documented as discussed here.

Member

jsirois commented Nov 23, 2016

This addresses the confusion caused by the jar goal being publicly documented as discussed here.

@stuhood stuhood merged commit fca742e into pantsbuild:master Nov 28, 2016

1 of 2 checks passed

coverage/coveralls Coverage pending from Coveralls.io
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jsirois jsirois deleted the jsirois:jsirois/jar/hide-task branch Nov 28, 2016

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

Un-publish the `jar` goal. (#4095)
This leaves the `jar.create` task installed and the `jar` goal
addressable from the command line, but no longer advertises the goal
since it's really not intended as-is for end users to invoke directly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment