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

Upgrade to go 1.21, replace golang.org/x/exp/slices with slices #4688

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Tiscs
Copy link

@Tiscs Tiscs commented Sep 1, 2024

Upgrade to go 1.21 to use slices package instead of golang.org/x/exp/slices package.

In the antlr4-go/antlr project, the go version declared in go.mod is already 1.22.

@jimidle
Copy link
Collaborator

jimidle commented Oct 11, 2024 via email

@skitt
Copy link

skitt commented Oct 11, 2024

I have already done this. It does not need a pr

It’s just confusing because it’s done in https://github.com/antlr4-go/antlr but not in https://github.com/antlr/antlr4/tree/dev/runtime/Go/antlr/v4 and the former says it’s a copy of the latter.

@jimidle
Copy link
Collaborator

jimidle commented Oct 11, 2024 via email

@skitt
Copy link

skitt commented Oct 11, 2024

Yes, but it referee to the build and release. You only use tagged versions

OK, sorry for the noise.

@Tiscs
Copy link
Author

Tiscs commented Oct 11, 2024

I have already done this. It does not need a pr

It’s just confusing because it’s done in https://github.com/antlr4-go/antlr but not in https://github.com/antlr/antlr4/tree/dev/runtime/Go/antlr/v4 and the former says it’s a copy of the latter.

I have previously created a PR in the antlr4-go/antlr project. but as mentioned in the project README.md, Do not submit PRs or any change requests to this repo, I closed that PR.

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.

3 participants