Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Prepopulate tstorage for the new version of EvmGasManager #93

Conversation

0xVolosnikov
Copy link
Collaborator

What ❔

Required for matter-labs/era-contracts#738

The logic of prepopulating storage is left for compatibility with the previous version of the contract

Why ❔

Checklist

  • PR title corresponds to the body of PR.
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via cargo fmt and checked with cargo clippy.

hedgar2017
hedgar2017 previously approved these changes Aug 26, 2024
@hedgar2017 hedgar2017 force-pushed the vv-prepopulate-tstorage-for-evmgasmanager branch from 7306db6 to 74478bf Compare August 26, 2024 22:29
@hedgar2017
Copy link
Collaborator

@0xVolosnikov please don't update this branch!
There is a potential bug in CI revealed here. Let us check it first.

@antonbaliasnikov
Copy link
Contributor

@0xVolosnikov please don't update this branch! There is a potential bug in CI revealed here. Let us check it first.

Hi guys,
thanks for finding this.

The issue is related to the case when we have a fork repository, and we would like to merge its other branch (not main), and main of the fork is not rebased from the upstream.

This resulted in the not rebased fork main to be in benchmarks and fail + an issue with integration tests defining the other branch.

I pushed a fix and am testing it now. I revert everything back when completed.

@antonbaliasnikov antonbaliasnikov force-pushed the vv-prepopulate-tstorage-for-evmgasmanager branch 2 times, most recently from 7921362 to 74478bf Compare August 27, 2024 09:09
@antonbaliasnikov
Copy link
Contributor

@0xVolosnikov , @hedgar2017 , CI is updated, all looks good to me now.

@hedgar2017 hedgar2017 merged commit 37d6f9f into matter-labs:main Aug 27, 2024
8 checks passed
@0xVolosnikov 0xVolosnikov deleted the vv-prepopulate-tstorage-for-evmgasmanager branch August 27, 2024 10:56
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.

3 participants