Skip to content
This repository has been archived by the owner on Feb 3, 2018. It is now read-only.

Get rid of Manifest.TestDependencies() entirely #140

Open
sdboyer opened this issue Jan 12, 2017 · 2 comments
Open

Get rid of Manifest.TestDependencies() entirely #140

sdboyer opened this issue Jan 12, 2017 · 2 comments

Comments

@sdboyer
Copy link
Owner

sdboyer commented Jan 12, 2017

Because the constraint decls no longer imply "requirement," having a separate method on Manifest for declaring test deps is utterly pointless. They should all be lumped in together, and #44 can deal with marking anything that comes in through tests as test-only.
@sdboyer sdboyer changed the title Get rid of TestDependencies() entirely Get rid of Manifest.TestDependencies() entirely Jan 12, 2017
@sdboyer
Copy link
Owner Author

sdboyer commented Feb 13, 2017

note to self (ish) - getting rid of this should mean that pcSliceToMap() can probably also go.

@fabulous-gopher fabulous-gopher mentioned this issue Apr 21, 2017
Closed
@fabulous-gopher
Copy link

This issue was moved to golang/dep#429

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sdboyer @fabulous-gopher