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

generated Go causes staticcheck errors #804

Closed
mcgaw opened this issue Feb 28, 2023 · 1 comment
Closed

generated Go causes staticcheck errors #804

mcgaw opened this issue Feb 28, 2023 · 1 comment
Labels
Go Go language support Wont Fix Out of scope of this project

Comments

@mcgaw
Copy link

mcgaw commented Feb 28, 2023

error var _ should have name of the form errFoo (ST1012) and at least one file in a package should have a package comment (ST1000) are emitted by the staticcheck tool.

There is a long quite tedious discussion about the use of "exclude" in staticcheck, but wondering if this is something that could be quite simple to fix?

In the meantime I've added a staticcheck.conf file to the generated output dir, which includes the line checks = ["all", "-ST1012","-ST1000"], but that's less than ideal.

@dominikh
Copy link

Staticcheck really shouldn't be flagging the blank identifier in ST1012; that's something I should fix.

@rodaine rodaine added Wont Fix Out of scope of this project Go Go language support labels Apr 20, 2023
@rodaine rodaine closed this as completed Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Go Go language support Wont Fix Out of scope of this project
Projects
None yet
Development

No branches or pull requests

3 participants