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

Updated Gopkg.lock / vendor #34

Merged
merged 1 commit into from
Jul 21, 2018
Merged

Updated Gopkg.lock / vendor #34

merged 1 commit into from
Jul 21, 2018

Conversation

mvisonneau
Copy link
Contributor

No description provided.

@mvisonneau mvisonneau changed the title Updated Gopkg.lock Updated Gopkg.lock / vendor Jul 20, 2018
@sevagh
Copy link
Owner

sevagh commented Jul 20, 2018

Is this just a general update of all the dependencies? Or was there a sync problem with the lock file and toml file?

@mvisonneau
Copy link
Contributor Author

Yep, just a general update. It compiles fine and still seems to work properly 👌

@mvisonneau
Copy link
Contributor Author

mvisonneau commented Jul 20, 2018

out of curiosity, is there any particular reason/concerns not to put vendor as part of a .gitignore ? 🤔

@sevagh
Copy link
Owner

sevagh commented Jul 20, 2018

https://github.com/golang/dep/blob/master/docs/FAQ.md#should-i-commit-my-vendor-directory

It's an additional safety measure in case my dependencies change, disappear, etc.

@mvisonneau
Copy link
Contributor Author

ah ok, cool, thanks for the info 👍

@sevagh
Copy link
Owner

sevagh commented Jul 21, 2018

Thanks for all the contributions. Are you using goat in production?

@sevagh sevagh merged commit c579fe8 into sevagh:master Jul 21, 2018
@mvisonneau mvisonneau deleted the gopkg branch July 21, 2018 13:02
@mvisonneau
Copy link
Contributor Author

yw! not yet but we are planning to! :)

sevagh pushed a commit that referenced this pull request Nov 21, 2018
sevagh pushed a commit that referenced this pull request Nov 28, 2018
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