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

runtime(go): minimum requirement go1.18 #4754

Closed
wants to merge 1 commit into from
Closed

Conversation

xiaost
Copy link

@xiaost xiaost commented Jan 10, 2025

golang.org/x/exp,
which requires newer go version,
is too heavy for previous implementation.

may duplicated with #4688 #4356 #4294, related #4663

@xiaost
Copy link
Author

xiaost commented Jan 10, 2025

We're proudly using antlr4, but we need to support users who're using go1.18.
The go runtime of antlr4 doesn't use any new features, and go1.18 is good enough.

I found that https://github.com/antlr4-go/antlr/ is using go1.22 while this repos is go1.20
Possibly it's caused by golang.org/x/exp

@xiaost
Copy link
Author

xiaost commented Jan 10, 2025

I'm checking the CI.
seems like if we have no dependency, go mod tidy won't generate go.sum and it causes the test runner java.io.FileNotFoundException

working on it.

@xiaost xiaost force-pushed the rm-x-exp-pkg branch 2 times, most recently from 888ca4d to 921fdfa Compare January 10, 2025 06:16
golang.org/x/exp,
which requires newer go version,
is too heavy for previous implementation

Signed-off-by: Kyle Xiao <[email protected]>
@xiaost
Copy link
Author

xiaost commented Jan 10, 2025

JFYI.
I also changed CI settings macos-12 -> macos-13
see: actions/runner-images#10721

@jimidle
Copy link
Collaborator

jimidle commented Jan 10, 2025 via email

@xiaost xiaost closed this Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants