-
Notifications
You must be signed in to change notification settings - Fork 91
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
🌱 Add envsubst and gomock to hack/tools/go.mod #207
🌱 Add envsubst and gomock to hack/tools/go.mod #207
Conversation
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.
please take care of the failed generate
test
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kashifest, wgslr 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 |
/test-integration |
/test-v1a3-centos-integration |
0ea1f00
to
a8c55f1
Compare
/test unit |
ea6a3e0
to
1360739
Compare
/test-integration |
/test-v1a3-integration |
/test-v1a3-integration |
/test-v1a3-centos-integration |
5 similar comments
/test-v1a3-centos-integration |
/test-v1a3-centos-integration |
/test-v1a3-centos-integration |
/test-v1a3-centos-integration |
/test-v1a3-centos-integration |
/test-integration |
This fixes Makefile targets for building the tool binaries. Prevent future removal of those entries by 'go mod tidy' by creating a fictitious import in tools.go. Use gomock version 1.5.0 to match the top-level go.mod file. Regenerate baremetal/mocks due to differences between 1.4.3 and 1.5.0 (generated methods are sorted).
1360739
to
ae2d06f
Compare
/test-integration |
Add a configuration file to ignore staticheck error SA1019 about deprecated types, which was not present when using older linter versions.
ae2d06f
to
48de806
Compare
/test-integration |
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
What this PR does / why we need it:
Fixes Makefile targets for building tool binaries, broken by a 'go mod tidy' run in a recent PR.