-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Verify vendor directory #14
Comments
Looks like the |
If the digest fields are missing it means an older |
Adding the script might be a too early now since it uses the |
/assign @pivotal-nader-ziada |
Thanks for taking this on @pivotal-nader-ziada ! We don't have prow working yet (#15) so commands like |
Ah that probably explains the problem - we should add something to our DEVELOPMENT.md requirements about the version of |
Expected Behavior
Running
dep ensure
(or whatever our DEVELOMENT.md says to run) should be a no-op on the repo unless deps have actually changed.Actual Behavior
When I run
dep ensure -v
on the repo, I get a whole slew of changes tovendor/
Steps to Reproduce the Problem
dep ensure -v
Additional Info
In the long run we'd like to use a script as part of CI to make sure vendor/ is kept in good shape on master. Potentially we can use:
The text was updated successfully, but these errors were encountered: