Dep: Use default prune settings during dep init

Created on 27 Nov 2017  路  8Comments  路  Source: golang/dep

Now that prune is (almost) absorbed into ensure, dep init should set some safe prune defaults on the generated manifest. init should also apply the prune options against vendor.

[prune]
  go-tests = true
  unused-packages = true

See https://github.com/golang/dep/pull/952 and https://github.com/golang/dep/pull/1226 for context.

help wanted init

Most helpful comment

definitely want to have this one before we release v0.4.0 - what dep init does will define a standard behavior that folks will follow. a sane default is key.

All 8 comments

I can pick this one up if no one else wants it. 馃榿

It's all yours 馃巵

definitely want to have this one before we release v0.4.0 - what dep init does will define a standard behavior that folks will follow. a sane default is key.

Hey folks, just wanted to drop a quick update. Still planning on getting to this one by the end of this upcoming week/weekend. My original plan was this past weekend but I've been sick 馃

If this blocking v0.4.0 and someone wants to get it done it before this week, feel free to take it. 馃

Feel better!

Also I've updated the OP with the desired default prune settings.

Excellent! Thank you. That was what I was thinking too -- non-go might be a little over the top for a default setting. 馃槂

@carolynvs we're just keeping this open until we merge the pending prune branch, right?

It's just how github works. It doesn't auto-close an issue when a PR is merged into the non-default branch. So once the release branch is merged, _then_ all the issues will be closed.

Was this page helpful?
0 / 5 - 0 ratings