-
Notifications
You must be signed in to change notification settings - Fork 101
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
🌱 bump ginkgo to v2.14.0 #1403
🌱 bump ginkgo to v2.14.0 #1403
Conversation
/test-centos-e2e-integration-main |
Dependabot tries to bump ginkgo in metal3-io#1401, metal3-io#1402, but the ginkgo is actually missing from the hack/tools go.mod and those then fail when "go get ..." is run and it dirties the go.mod/go.sum in hack/tools. Kustomize build then fails on the dirty go.sum file. Added the ginkgo to hack/tools/go.mod as well. Signed-off-by: Tuomo Tanskanen <[email protected]>
d18ebb6
to
c22be70
Compare
/test-centos-e2e-integration-main |
1 similar comment
/test-centos-e2e-integration-main |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kashifest The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Dependabot tries to bump ginkgo in #1401, #1402, but separately, which causes our Makefile "e2e-tests" go haywire. Ginkgo is actually missing from hack/tools/go.mod, and when "go get ginkgo..." is run, it dirties the go.sum/go.mod and the next build for kustomize explode.
Thus, add ginkgo to hack/tools/go.mod as well.