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

x/tools/gopls: The analyzers documentation doesn't mention some analyzers #52874

Open
pjweinb opened this issue May 12, 2022 · 3 comments
Open

x/tools/gopls: The analyzers documentation doesn't mention some analyzers #52874

pjweinb opened this issue May 12, 2022 · 3 comments
Labels
Documentation gopls Tools
Milestone

Comments

@pjweinb
Copy link

@pjweinb pjweinb commented May 12, 2022

gopls version

What did you do?

I read x/tools/gopls/doc/analyzers.md to try to find the setting that would turn off useless messages labelled QF1003.

What did you expect to see?

Some mention of QF1003, or a hint about where to find information about it.

What did you see instead?

nothing.

Editor and settings

@gopherbot gopherbot added Tools gopls labels May 12, 2022
@gopherbot gopherbot added this to the Unreleased milestone May 12, 2022
@pjweinb
Copy link
Author

@pjweinb pjweinb commented May 13, 2022

The code at x/tools/gopls/docgenerate.go:101 says "// Don't yet add staticcheck analyzers".

@dominikh
Copy link
Member

@dominikh dominikh commented May 13, 2022

In the meantime, all of Staticcheck's analyzers are documented at https://staticcheck.io/docs/checks/.

@findleyr
Copy link
Contributor

@findleyr findleyr commented May 15, 2022

I think we should just clearly link to staticcheck docs.

@findleyr findleyr removed this from the Unreleased milestone May 15, 2022
@findleyr findleyr added this to the gopls/later milestone May 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation gopls Tools
Projects
None yet
Development

No branches or pull requests

4 participants