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

remove golangci-lint #428

Closed
wants to merge 1 commit into from
Closed

Conversation

rminnich
Copy link
Contributor

It has useful recommendations, but the output is
not great, errors with no indications as to which
file it found the error; and it ignores the go version specified in go.mod.

It has useful recommendations, but the output is
not great, errors with no indications as to which
file it found the error; and it ignores the go version
specified in go.mod.

Signed-off-by: Ronald G. Minnich <[email protected]>
@orangecms
Copy link
Collaborator

We'll also need to remove the integration in the GitHub project settings.
I'll look into that.

@orangecms
Copy link
Collaborator

Let's check also how it behaves locally and if newer versions of tje GitHub integration are available. It may just be a simpler matter of a version bump.
We might be able to supply the Go version, let's see about that as well.

@rminnich
Copy link
Contributor Author

sounds good, whatever seems best. I'm just getting tired of its not quite helpful recommendations. :-)

@orangecms
Copy link
Collaborator

See #429 - looking good!

The Go version to use is not read from go.mod intentionally.
It is the version that is selected/installed in the system running the command, just as if you ran it locally.

The docs https://github.com/golangci/golangci-lint-action tell how to specify both the linter version and the Go version.

@rminnich rminnich closed this Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants