Closed rexagod closed 4 days ago
Hmm I can't get go vet ./...
to fail on my machine :-/
Strange, are you on go1.21
? I can reproduce this in a container if that's better.
How about making use of golangci's govet https://golangci-lint.run/usage/linters/ and enable it for tests? that way we're sure it'll run in CI as well. (for now we skip all the tests https://github.com/openshift/cluster-monitoring-operator/blob/953011b1a4cdd66e2b42e16facca4ccc65079b7e/.golangci.yaml#L3 but I think we can have better filtering: we want some linters to run on tests as well)
@rexagod: This pull request references MON-3850 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.16.0" version, but no target version was set.
Issues go stale after 90d of inactivity.
Mark the issue as fresh by commenting /remove-lifecycle stale
.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen
.
If this issue is safe to close now please do so with /close
.
/lifecycle stale
/remove-lifecycle stale /cc @machine424
Pushed the suggested changes.
(bump)
/lgtm Thanks for this.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: machine424, rexagod, simonpasquier
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@rexagod: all tests passed!
Full PR test history. Your PR dashboard.
[ART PR BUILD NOTIFIER]
Distgit: cluster-monitoring-operator This PR has been included in build cluster-monitoring-operator-container-v4.18.0-202410162014.p0.g82de4b7.assembly.stream.el9. All builds following this will include this PR.
Adds an explicit
vet
target and also integrates it into theverify
flow to catch any such violations within the source code.