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

Verify vendor directory #14

Closed
bobcatfish opened this issue Sep 7, 2018 · 6 comments
Closed

Verify vendor directory #14

bobcatfish opened this issue Sep 7, 2018 · 6 comments
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.

Comments

@bobcatfish
Copy link
Collaborator

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 to vendor/

Steps to Reproduce the Problem

  1. Checkout repo
  2. Install go dep
  3. Run 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:

@bobcatfish bobcatfish added help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. labels Sep 7, 2018
@nader-ziada
Copy link
Member

Looks like the Gopkg.lock file is missing the digest field which gets added by running dep ensure

@dprotaso
Copy link
Contributor

If the digest fields are missing it means an older dep was used. We should be using the latest 0.5.0.

@nader-ziada
Copy link
Member

Adding the script might be a too early now since it uses the test-infra repo but there is no code using that yet. In serving, its used by the e2e tests

@nader-ziada
Copy link
Member

/assign @pivotal-nader-ziada

@bobcatfish
Copy link
Collaborator Author

Thanks for taking this on @pivotal-nader-ziada ! We don't have prow working yet (#15) so commands like /assign don't yet do anything 😅

@bobcatfish
Copy link
Collaborator Author

If the digest fields are missing it means an older dep was used. We should be using the latest 0.5.0.

Ah that probably explains the problem - we should add something to our DEVELOPMENT.md requirements about the version of dep.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.
Projects
None yet
Development

No branches or pull requests

3 participants