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

Apply upgrades after state at block #999

Merged
merged 15 commits into from
Nov 30, 2023
Merged

Conversation

ceyonur
Copy link
Collaborator

@ceyonur ceyonur commented Nov 16, 2023

Why this should be merged

Continues to fix other tracer APIs suffering from similar issue as in #976

Closes: #992

How this works

Calls ApplyUpgrades after StateAtBlock if necessary.

How this was tested

Added unit tests.

How is this documented

No need

@ceyonur ceyonur added the bug Something isn't working label Nov 16, 2023
@ceyonur ceyonur added this to the v0.5.10 milestone Nov 16, 2023
@ceyonur ceyonur self-assigned this Nov 16, 2023
@ceyonur ceyonur requested review from marun and nytzuga November 16, 2023 17:27
eth/tracers/api_test.go Outdated Show resolved Hide resolved
eth/state_accessor.go Outdated Show resolved Hide resolved
eth/tracers/api_test.go Outdated Show resolved Hide resolved
eth/tracers/api_test.go Outdated Show resolved Hide resolved
eth/tracers/api.go Outdated Show resolved Hide resolved
eth/tracers/api.go Outdated Show resolved Hide resolved
eth/tracers/api.go Outdated Show resolved Hide resolved
ceyonur and others added 5 commits November 17, 2023 12:36
Co-authored-by: Darioush Jalali <darioush.jalali@avalabs.org>
Signed-off-by: Ceyhun Onur <ceyhunonur54@gmail.com>
…ubnet-evm into apply-upgrades-state-at-block
@ceyonur ceyonur requested review from marun and darioush November 18, 2023 11:11
darioush
darioush previously approved these changes Nov 29, 2023
eth/state_accessor.go Outdated Show resolved Hide resolved
eth/tracers/api_test.go Outdated Show resolved Hide resolved
marun
marun previously approved these changes Nov 29, 2023
@ceyonur ceyonur dismissed stale reviews from marun and darioush via ba22f02 November 29, 2023 23:21
@ceyonur ceyonur requested review from marun and darioush November 29, 2023 23:21
@ceyonur ceyonur enabled auto-merge (squash) November 29, 2023 23:22
@ceyonur ceyonur merged commit 2113943 into master Nov 30, 2023
8 checks passed
@ceyonur ceyonur deleted the apply-upgrades-state-at-block branch November 30, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Apply Upgrades after StateAtBlock
3 participants