Skip to content

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 <xiaost7@gmail.com>
@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