Skip to content

Update golang.org/x/tools to v0.31.0 and go to 1.24.1#108

Merged
dmarkham merged 1 commit intodmarkham:masterfrom
graphman65:feature/bump-go-tools-version
Jan 21, 2026
Merged

Update golang.org/x/tools to v0.31.0 and go to 1.24.1#108
dmarkham merged 1 commit intodmarkham:masterfrom
graphman65:feature/bump-go-tools-version

Conversation

@graphman65
Copy link
Copy Markdown
Contributor

After updating our project version to 1.24 we had to bump enumer to v1.5.11 and it started failing for all our usages with no value for constant <variable-name>.

After bumping x/tools to v0.31.0 on enumer it works fine again (had to upgrade go too at the same time as it's required for the new version of x/tools.

@chidionuekwusi-nala
Copy link
Copy Markdown

+1

1 similar comment
@ansssu
Copy link
Copy Markdown

ansssu commented May 28, 2025

+1

@EngHabu
Copy link
Copy Markdown

EngHabu commented Sep 18, 2025

When is this going to be merged?

@omarayad-telda
Copy link
Copy Markdown

@dmarkham @samiam2013 Is there a timeline for this PR to be merged? This library has been a ton of help for our org but it is currently capping our golang version and we're unable to move forward – any updates would be very appreciated 🙏

@samiam2013
Copy link
Copy Markdown
Collaborator

I get so many notification emails from github, I usually miss these. Didn't know a tool change was blocking on enumer. I'll try to get to this soon, but things are abnormally busy around here. Sorry!

@SVilgelm
Copy link
Copy Markdown

SVilgelm commented Jan 9, 2026

Hey @samiam2013, Could you please come back to this PR?

@SVilgelm
Copy link
Copy Markdown

@dmarkham Hi, could you please review this PR? This fix can simplify our CI a lot, we are forced to manually clone a fork with this fix and install from the fork instead of simple go install :)

@dmarkham dmarkham merged commit 17a8006 into dmarkham:master Jan 21, 2026
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.

8 participants