-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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/go/analysis: migrate golint to use go/analysis #36869
Labels
FeatureRequest
FrozenDueToAge
gopls
Issues related to the Go language server, gopls.
Tools
This label describes issues relating to any tools in the x/tools repository.
Comments
gopherbot
added
the
Tools
This label describes issues relating to any tools in the x/tools repository.
label
Jan 29, 2020
cc @hyangah, who I think has done some work on this. I think golist should stay in a separate repository. We could put the passes in tools (though I'd prefer not to) but the binary itself should not be in tools. |
/cc @ridersofrohan |
Relevant: #38968 |
#38968 is accepted and golint is deprecated and being frozen this week - closing. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
FeatureRequest
FrozenDueToAge
gopls
Issues related to the Go language server, gopls.
Tools
This label describes issues relating to any tools in the x/tools repository.
This is a key missing piece in the
gopls
analysis suite, and it has come up repeatedly as an issue for editor clients (see microsoft/vscode-go#2517 for a recent example). Creating a tracking issue here sincegolint
has its own issue tracker and its own issue for this (golang/lint#463).Something worth considering is whether this means that
golint
will be moved intox/tools
or if it should remain its separate repository./cc @matloob
The text was updated successfully, but these errors were encountered: