Skip to content

Conversation

@Wodann
Copy link
Member

@Wodann Wodann commented Jan 30, 2026

  • Because this PR includes a bug fix, relevant tests have been included.
  • Because this PR includes a new feature, the change was previously discussed on an Issue or with someone from the team.
  • I didn't do anything of this.

Community plugins were using the original MinimalEthereumJsVm instance, so instead of replacing it, we now change its internal stateManager using an updateProvider function, which was the only thing that needs an updated provider.

This should fix the secondary issue reported in #7834.

Community plugins were using the original `MinimalEthereumJsVm` instance,
so instead of replacing it, we now change its internal `stateManager`
which was the only thing that needs an updated provider.
@Wodann Wodann requested a review from a team January 30, 2026 17:02
@Wodann Wodann self-assigned this Jan 30, 2026
@Wodann Wodann added the no docs needed This PR doesn't require links to documentation label Jan 30, 2026
@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2026

🦋 Changeset detected

Latest commit: cd8db29

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
hardhat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@schaable schaable self-requested a review February 9, 2026 15:50
Copy link
Member

@schaable schaable left a comment

Choose a reason for hiding this comment

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

LGTM

@Wodann Wodann merged commit f6d5437 into v2 Feb 10, 2026
323 of 345 checks passed
@Wodann Wodann deleted the fix/issue-7834 branch February 10, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no docs needed This PR doesn't require links to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants