Skip to content

Conversation

@StephenButtolph
Copy link
Contributor

Why this should be merged

Currently, it is not possible to use avalanchego as a dependency, because the go.mod references a grafted coreth version that doesn't exist.

How this works

The coreth version in the go.mod isn't actually used for local builds, because of the replace directive. However, when other libraries attempt to fetch avalanchego, they do not honor the replace directive. This means that they will attempt to fetch the version of coreth in our go.mod, which doesn't exist.

How this was tested

I was able to update subnet-evm to this version of avalanchego after updating this reference.

Need to be documented in RELEASES.md?

no

Copilot AI review requested due to automatic review settings December 1, 2025 18:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the coreth dependency version in go.mod from a non-existent grafted version to a valid commit hash, enabling avalanchego to be used as a dependency by other projects.

Key Changes:

  • Updated coreth version reference from v0.16.0-rc.0 to a valid commit hash v0.0.0-20251201152440-a263a7cedcd0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@JonathanOppenheimer JonathanOppenheimer left a comment

Choose a reason for hiding this comment

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

laughing at the PR name

@StephenButtolph StephenButtolph added this pull request to the merge queue Dec 1, 2025
Merged via the queue into master with commit badbe80 Dec 1, 2025
40 checks passed
@StephenButtolph StephenButtolph deleted the update-coreth-to-recent-hash branch December 1, 2025 19:31
@github-project-automation github-project-automation bot moved this to Done 🎉 in avalanchego Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done 🎉

Development

Successfully merging this pull request may close these issues.

4 participants