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

Question about commit the vendor directory #547

Closed
iblancasa opened this issue Oct 7, 2017 · 2 comments
Closed

Question about commit the vendor directory #547

iblancasa opened this issue Oct 7, 2017 · 2 comments

Comments

@iblancasa
Copy link

I read this in the documentation:

Read over the contents of vendor/ and make sure it looks reasonable. Then commit the Godeps/ and vendor/ directories to version control.

Why it is a good practice to commit the vendor directory? Usually this is something bad in other languages.

I didn't found information about this topic.

Thank you for your time.

@Ortham
Copy link

Ortham commented Oct 17, 2017

I'm wondering this too, since godep restore and godep get seem to have no problem downloading and installing the dependencies specified in Godeps.json. If we don't need offline builds, do we really need to commit the vendor/ folder? Is something going to blow up in my face if I don't?

The FAQ gives the reasoning why the godep authors think vendor/ should be checked in, but some clarity that this is (seems to be?) merely an opinion and not a technical requirement would go a long way. For example, the readme currently has

Then commit the Godeps/ and vendor/ directories to version control.

Is the following alternative valid?

Then commit the Godeps/ directory to version control. The godep authors also recommend committing the vendor/ directory to allow offline builds and reduce reliance on the availability of external systems.

@freeformz
Copy link

I am closing this issue because the repo is going to be archived.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants