Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove vendor folder entirely #83

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

palfrey
Copy link
Contributor

@palfrey palfrey commented Aug 23, 2017

Fixes #81

@jgsqware
Copy link
Owner

jgsqware commented Sep 8, 2017

Partly agree with that. problem is that in the glide.yaml some lin are not pin to a tag or HASH. So we are not sure that the next glide install/update will have the same behavior.

So could you pin the least library in the yaml file? Than we can remove the vendor folder.

@palfrey
Copy link
Contributor Author

palfrey commented Sep 8, 2017

I've locked down every dependency from glide.yaml that wasn't already locked down. As I was also seeing Lock file may be out of date. Hash check of YAML failed. You may need to run 'update' from glide install I've updated the glide.lock as well.

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

Successfully merging this pull request may close these issues.

2 participants