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

Jump to bottom

x/tools/gopls: inconsistent semantic token types for typeParameter in function recevier #57619

Closed
clouky-grey opened this issue Jan 5, 2023 · 1 comment
Closed

x/tools/gopls: inconsistent semantic token types for typeParameter in function recevier #57619

clouky-grey opened this issue Jan 5, 2023 · 1 comment
Assignees
@pjweinb
Labels
gopls Issues related to the Go language server, gopls. Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
gopls/later

Comments

@clouky-grey
Copy link

As example, the struct "SMap" has generic typeParameter "K" and "V".
iShot_2023-01-05_10 35 53
but in receiver, it check "K" and "V" as variable
iShot_2023-01-05_10 36 22
iShot_2023-01-05_10 36 36
i think it should be typeParameter like others
iShot_2023-01-05_10 36 53
iShot_2023-01-05_10 38 00
@gopherbot gopherbot added Tools This label describes issues relating to any tools in the x/tools repository. gopls Issues related to the Go language server, gopls. labels Jan 5, 2023
@gopherbot gopherbot added this to the Unreleased milestone Jan 5, 2023
@jamalc jamalc assigned jamalc and pjweinb and unassigned jamalc Jan 5, 2023
@dle8 dle8 modified the milestones: Unreleased, gopls/later Jan 19, 2023
@gopherbot
Copy link

Change https://go.dev/cl/463316 mentions this issue: gopls/semantic: report type parameters in the type of a receiver

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pjweinb pjweinb

Labels
gopls Issues related to the Go language server, gopls. Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Milestone
gopls/later
Development

No branches or pull requests
5 participants
@pjweinb @gopherbot @jamalc @clouky-grey @dle8