Skip to content

Conversation

ARR4N
Copy link
Collaborator

@ARR4N ARR4N commented Sep 22, 2025

Why this should be merged

Version increase across all ava-labs EVM-related code.

How this works

Bump to 1.24.8 in both go.mod files + requisite version increase for golangci-lint.

How this was tested

Existing tests.

@ARR4N
Copy link
Collaborator Author

ARR4N commented Sep 25, 2025

Unnecessary and causing too many flow-on effects.

@ARR4N ARR4N closed this Sep 25, 2025
@alarso16 alarso16 reopened this Oct 10, 2025
@alarso16 alarso16 marked this pull request as ready for review October 10, 2025 18:17
@alarso16
Copy link

This was deemed unnecessary earlier, but to add firewood in this repository, we must update to a modern version of Go

@ARR4N
Copy link
Collaborator Author

ARR4N commented Oct 13, 2025

Thanks for fixing the vet issues @alarso16. I've added this PR to #128 so they don't cause merge conflicts when we do a geth sync.

Copy link

@alarso16 alarso16 left a comment

Choose a reason for hiding this comment

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

Approving on behalf of @ARR4N

@alarso16 alarso16 merged commit 51afede into main Oct 13, 2025
12 checks passed
@alarso16 alarso16 deleted the arr4n/go-1.24 branch October 13, 2025 18:07
ARR4N added a commit that referenced this pull request Oct 18, 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