Skip to content

Conversation

@markya0616
Copy link

No description provided.

@markya0616 markya0616 force-pushed the feature/init_state branch from cc5712b to 1c71130 Compare April 3, 2018 03:14
@markya0616 markya0616 force-pushed the feature/init_state branch from 1c71130 to a4bbfd6 Compare April 3, 2018 03:50
yysung1123 pushed a commit that referenced this pull request Oct 24, 2025
> [!NOTE]
> This will be merged into `main` once the current target branch has
been merged.

## Why this should be merged

My original implementation was back to front and it's been bugging me.

## How this works

Originally `params.LibEVMVersion` was a `var` (because it needed to be
computed) and the test used a `const`. This change simply inverts the
two and moves some code around without any change in logic.

I bumped the minor version to 2 (a no-op when not on a release branch)
to bring it in line with the latest release candidate and to avoid
forgetting to do so when performing an actual release.

## How this was tested

Unit test of version-string constant.
yysung1123 pushed a commit that referenced this pull request Oct 24, 2025
#163 was removed by mistake in #162 so this re-adds it

Co-authored-by: Arran Schlosberg <519948+ARR4N@users.noreply.github.com>
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