Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Implement fetching internal transactions from callTracer #885

Merged
merged 11 commits into from
Jul 18, 2023

Conversation

carterqw2
Copy link

@carterqw2 carterqw2 commented Mar 15, 2023

Description

Cherry-picks the following commits from upstream:
https://github.com/blockscout/blockscout/pull/6721/commits
blockscout@b11e10e

This adds support for callTracer and makes it a default tracer instead of the custom js tracer. This branch needs to be tested along with the native tracers PR in the blockchain client.

Other changes

No.

Tested

There is a test included.

@github-actions
Copy link

github-actions bot commented Mar 15, 2023

Unit Test Results

       6 files  ±0     366 suites  ±0   4m 56s ⏱️ + 1m 21s
2 650 tests ±0  2 576 ✔️ ±0  74 💤 ±0  0 ±0 
2 664 runs  ±0  2 588 ✔️ ±0  76 💤 ±0  0 ±0 

Results for commit 0e6a724. ± Comparison against base commit 5c6f431.

♻️ This comment has been updated with latest results.

@carterqw2 carterqw2 force-pushed the carterqw2/call-tracer branch 2 times, most recently from f96a366 to a74cbb4 Compare March 15, 2023 12:32
@carterqw2 carterqw2 marked this pull request as ready for review March 15, 2023 12:39
@carterqw2 carterqw2 requested a review from a team March 15, 2023 12:39
@carterqw2 carterqw2 force-pushed the carterqw2/call-tracer branch from 3e2e0e0 to 1863b52 Compare April 14, 2023 12:49
@carterqw2 carterqw2 force-pushed the carterqw2/call-tracer branch from 8909347 to f241e2e Compare June 22, 2023 14:42
@rkachowski rkachowski force-pushed the carterqw2/call-tracer branch from f241e2e to c822503 Compare July 3, 2023 16:18
@rkachowski rkachowski self-assigned this Jul 6, 2023
@rkachowski rkachowski enabled auto-merge July 18, 2023 08:14
@rkachowski rkachowski merged commit e2e2982 into master Jul 18, 2023
@rkachowski rkachowski deleted the carterqw2/call-tracer branch July 18, 2023 08:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants