Skip to content

refactor: simplify single UT for OpSpecId compatibility.#2216

Merged
rakita merged 1 commit intobluealloy:mainfrom
jackwener:remove
Mar 17, 2025
Merged

refactor: simplify single UT for OpSpecId compatibility.#2216
rakita merged 1 commit intobluealloy:mainfrom
jackwener:remove

Conversation

@jackwener
Copy link
Contributor

simplify UT for OpSpecId compatibility

/// If we don't expect parent blocks to be committed or unwinded from database, this option
/// should be used.
Yes,
/// Doesn't check original values, see the implementation of [BundleState::into_plain_state] for
Copy link
Contributor Author

@jackwener jackwener Mar 15, 2025

Choose a reason for hiding this comment

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

into_plain_state is deprecated

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 15, 2025

CodSpeed Performance Report

Merging #2216 will not alter performance

Comparing jackwener:remove (ce214d4) with main (f742477)

Summary

✅ 8 untouched benchmarks

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

Cool!

@rakita rakita merged commit e1054d1 into bluealloy:main Mar 17, 2025
29 checks passed
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