-
Bug
-
Resolution: Unresolved
-
Undefined
-
None
-
4.21
-
None
-
Quality / Stability / Reliability
-
False
-
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
-
None
Description of problem:
In my PR for updating dependencies, I noticed that golint was failing with a deprecated function in the integration tests:
The note says:
testscript.RunMain is deprecated: use [Main], as the only reason for returning exit codes was to collect full code coverage, which Go does automatically now: https://go.dev/blog/integration-test-coveragedeprecateddefault
I gave a quick shot at fixing it and also asked Claude, but it was not obvious how to use Main instead of RunMain. So instead I'm putting a nolint on the PR and we can fix later.
Version-Release number of selected component (if applicable):
main
How reproducible:
Steps to Reproduce:
go-lint with https://github.com/openshift/installer/pull/10060 or on main once that marges
Actual results:
Expected results:
Remove the //nolint and go-fmt should pass
Additional info: