Update target scope docs #4553

Merged
merged 1 commit into from May 5, 2017

Conversation

Projects
None yet
2 participants
@traviscrawford
Member

traviscrawford commented May 5, 2017

Problem

In #4544 we describe an issue where the test scope is not documented, and the docs do not let users know that multiple scopes can be specified for a target. In particular, it's not clear in the existing docs how to emulate the maven provided scope.

Solution

Add user documentation about the test scope, and how to emulate the Maven
provided scope in Pants.

Result

Users should be able to emulate the maven provided scope in pants.

Update target scope docs
Add user documentation about the `test` scope, and how to emulate the Maven
`provided` scope in Pants.
@traviscrawford

This comment has been minimized.

Show comment
Hide comment
@traviscrawford

traviscrawford May 5, 2017

Member

I generated the site locally and the formatting looks good. Happy to make any substance or style changes if you have suggestions!

Member

traviscrawford commented May 5, 2017

I generated the site locally and the formatting looks good. Happy to make any substance or style changes if you have suggestions!

@jsirois

jsirois approved these changes May 5, 2017

Thanks Travis!

@jsirois jsirois merged commit 1687a6f into pantsbuild:master May 5, 2017

1 check passed

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

@traviscrawford traviscrawford deleted the traviscrawford:travis/update-scope-docs branch May 9, 2017

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