Skip to content

Conversation

@nebasuke
Copy link
Member

Bump EDR to 0.8.0.

This fixes the following issues:

Minor Changes

  • NomicFoundation/edr@af26624: Improved provider initialization performance by passing build info as buffer which avoids FFI copy overhead

Patch Changes

@vercel
Copy link

vercel bot commented Jan 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hardhat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 31, 2025 9:25am

@changeset-bot
Copy link

changeset-bot bot commented Jan 30, 2025

🦋 Changeset detected

Latest commit: edb1490

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

Copy link
Member

@agostbiro agostbiro left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM, but I'd defer to @kanej to approve

try {
for (const buildInfoFile of buildInfoFiles) {
const buildInfo = await fsExtra.readJson(buildInfoFile);
if (semver.gte(buildInfo.solcVersion, FIRST_SOLC_VERSION_SUPPORTED)) {
Copy link
Member

Choose a reason for hiding this comment

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

config: HardhatNetworkProviderConfig,
loggerConfig: LoggerConfig,
tracingConfig?: TracingConfig
tracingConfig?: TracingConfigWithBuffers
Copy link
Member

Choose a reason for hiding this comment

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

I was initially worried about this move as it is changing the API, but EdrProviderWrapper is in internal and is not exposed to the user. The only place create is invoked, uses makeTracingConfig which has had its return type altered as well.

Put another way this signature change looks safe to me.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks John!

@nebasuke nebasuke merged commit f208510 into main Jan 31, 2025
133 checks passed
@nebasuke nebasuke deleted the chore/bump-edr-0.8.0 branch January 31, 2025 13:53
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants