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.

Sign up for GitHub

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

refactor(evm): remove double JSON escaping of `EventEthereumTx` logs #2188

Merged

k-yang merged 7 commits into main from refactor/evm/logs

Feb 4, 2025

Merged

refactor(evm): remove double JSON escaping of `EventEthereumTx` logs #2188

Merge branch 'main' into refactor/evm/logs

Codecov / codecov/patch succeeded Feb 4, 2025 in 1s

56.09% of diff hit (target 0.00%)

View this Pull Request on Codecov

56.09% of diff hit (target 0.00%)

Annotations

Check warning on line 226 in eth/rpc/backend/utils.go

codecov / codecov/patch

eth/rpc/backend/utils.go#L226

Added line #L226 was not covered by tests

Check warning on line 230 in eth/rpc/backend/utils.go

codecov / codecov/patch

eth/rpc/backend/utils.go#L230

Added line #L230 was not covered by tests

Check warning on line 255 in eth/rpc/backend/utils.go

codecov / codecov/patch

eth/rpc/backend/utils.go#L254-L255

Added lines #L254 - L255 were not covered by tests

Check warning on line 40 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L40

Added line #L40 was not covered by tests

Check warning on line 45 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L45

Added line #L45 was not covered by tests

Check warning on line 54 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L54

Added line #L54 was not covered by tests

Check warning on line 59 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L59

Added line #L59 was not covered by tests

Check warning on line 68 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L68

Added line #L68 was not covered by tests

Check warning on line 73 in x/evm/events.go

codecov / codecov/patch

x/evm/events.go#L73

Added line #L73 was not covered by tests

Check warning on line 100 in x/evm/keeper/msg_server.go

codecov / codecov/patch

x/evm/keeper/msg_server.go#L100

Added line #L100 was not covered by tests

Check warning on line 52 in x/evm/logs.go

codecov / codecov/patch

x/evm/logs.go#L51-L52

Added lines #L51 - L52 were not covered by tests

Check warning on line 61 in x/evm/logs.go

codecov / codecov/patch

x/evm/logs.go#L61

Added line #L61 was not covered by tests

Check warning on line 70 in x/evm/logs.go

codecov / codecov/patch

x/evm/logs.go#L70

Added line #L70 was not covered by tests

Check warning on line 76 in x/evm/logs.go

codecov / codecov/patch

x/evm/logs.go#L76

Added line #L76 was not covered by tests

View more details on Codecov