Skip to content

Updated Go Lexer/Parser for Go Lang 1.20#7269

Open
lkishalmi wants to merge 1 commit intoapache:masterfrom
lkishalmi:update-go.lang-1.20
Open

Updated Go Lexer/Parser for Go Lang 1.20#7269
lkishalmi wants to merge 1 commit intoapache:masterfrom
lkishalmi:update-go.lang-1.20

Conversation

@lkishalmi
Copy link
Copy Markdown
Contributor

Updated the GoLang grammar from the ANTLR4 Repo, it supports Go 1.20 which has a new ~ operator.

@lkishalmi lkishalmi added the Go Go Language label Apr 14, 2024
@lkishalmi lkishalmi added this to the NB22 milestone Apr 14, 2024
golang.grammar.version=bcc9ec1b35e4ede2f90abdb8713a56315a634a92

# Simple language update can result too many signature changes in the generated code.
sigtest.skip.gen=true
Copy link
Copy Markdown
Contributor

@matthiasblaesing matthiasblaesing Apr 15, 2024

Choose a reason for hiding this comment

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

Why drop the pretense that this is module has an API and require that users make dependencies implementation dependencies?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Agreed! Although I'm also curious why this is a module at all?

@ebarboni ebarboni modified the milestones: NB22, NB23 Apr 19, 2024
@mbien mbien modified the milestones: NB23, NB24 Jul 16, 2024
@mbien
Copy link
Copy Markdown
Member

mbien commented Jul 17, 2024

fyi: i moved the milestone to NB 24 for now. (but we could squeeze this one in too if the comments are addressed since the change is isolated to go)

@mbien
Copy link
Copy Markdown
Member

mbien commented Oct 11, 2024

@lkishalmi dropping the milestone since it has been bumped a few times already.

If you refresh this PR, feel free to add the a milestone again, there would be also still enough time to get this into NB 24 as of today.

@mbien mbien removed this from the NB24 milestone Oct 11, 2024
@mbien mbien added the ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) label Oct 11, 2024
@mbien
Copy link
Copy Markdown
Member

mbien commented Apr 14, 2026

lets try to get this in for NB 31 since this is a small one. adding milestone again ;)

@mbien mbien added this to the NB31 milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Go Go Language

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants