Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Should prune remove config files from other dep tools? #1631

Closed
carolynvs opened this issue Feb 5, 2018 · 2 comments
Closed

Should prune remove config files from other dep tools? #1631

carolynvs opened this issue Feb 5, 2018 · 2 comments
Labels

Comments

@carolynvs
Copy link
Collaborator

From @allenpetersen

As of v0.4.1 having prune set to unused-packages or non-go will remove manifest files from other tools, I've tested this with Godep.json files, I assume other tools have similar issues. Should prune be aware of the metadata files used by the Importers and prevent them from being deleted?

@carolynvs
Copy link
Collaborator Author

Once a dependency is vendored, I am unaware of any use for their configuration files. When dep starts taking in other pkg mgmt tools config files into account during dep ensure, dep will read the config file directly from the repo, not the vendor directory.

Do you have a use-case that would require them remaining in vendor?

@mvdan
Copy link
Member

mvdan commented Sep 4, 2020

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!

@mvdan mvdan closed this as completed Sep 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants