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

Go: Fix type param declarations in AST viewer #17794

Merged
merged 3 commits into from
Oct 18, 2024

Conversation

owen-mc
Copy link
Contributor

@owen-mc owen-mc commented Oct 17, 2024

type param declarations were just floating, not part of the tree rooted in at the file node. This is easily fixed.

Should this have a change note? src or lib?

Note that we currently don't extract any kind of type parameter declaration for methods on generic types. We probably ought to, for consistency. But for now it is correct that the tests don't show anything in that case.

@owen-mc owen-mc requested a review from a team as a code owner October 17, 2024 14:42
@github-actions github-actions bot added the Go label Oct 17, 2024
smowton
smowton previously approved these changes Oct 17, 2024
Copy link
Contributor

@smowton smowton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks plausible. Probably yes put a change note since the VSCode AST viewer exposes this just enough to be noteworthy. I'd say the note goes under lib.

@owen-mc
Copy link
Contributor Author

owen-mc commented Oct 18, 2024

@smowton I've added a change note. Please re-review.

@owen-mc owen-mc merged commit f3abe54 into github:main Oct 18, 2024
14 checks passed
@owen-mc owen-mc deleted the go/ast-viewer-typeparamdecl branch October 18, 2024 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants