Skip to content

feat(rpc): perform js inspector on spawn_with_call_at async tracing task#3957

Merged
mattsse merged 2 commits intoparadigmxyz:mainfrom
tcoratger:js-inspector
Jul 27, 2023
Merged

feat(rpc): perform js inspector on spawn_with_call_at async tracing task#3957
mattsse merged 2 commits intoparadigmxyz:mainfrom
tcoratger:js-inspector

Conversation

@tcoratger
Copy link
Contributor

Should solve #3951.

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pedantic style nit

@mattsse mattsse linked an issue Jul 27, 2023 that may be closed by this pull request
@mattsse mattsse added C-perf A change motivated by improving speed, memory usage or disk footprint A-rpc Related to the RPC implementation labels Jul 27, 2023
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #3957 (2c9845f) into main (f98b152) will increase coverage by 0.00%.
Report is 3 commits behind head on main.
The diff coverage is 0.00%.

Impacted file tree graph

Files Changed Coverage Δ
crates/rpc/rpc/src/debug.rs 11.75% <0.00%> (-0.03%) ⬇️

... and 11 files with indirect coverage changes

Flag Coverage Δ
integration-tests 15.54% <0.00%> (-0.02%) ⬇️
unit-tests 64.55% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
reth binary 27.18% <ø> (ø)
blockchain tree 83.04% <ø> (ø)
pipeline 89.82% <ø> (ø)
storage (db) 74.30% <ø> (+0.01%) ⬆️
trie 94.70% <ø> (ø)
txpool 46.56% <ø> (+0.56%) ⬆️
networking 77.65% <ø> (-0.05%) ⬇️
rpc 58.72% <0.00%> (-0.03%) ⬇️
consensus 64.46% <ø> (ø)
revm 33.68% <ø> (ø)
payload builder 6.61% <ø> (ø)
primitives 87.81% <ø> (-0.10%) ⬇️

@mattsse mattsse added this pull request to the merge queue Jul 27, 2023
Merged via the queue into paradigmxyz:main with commit 717bad8 Jul 27, 2023
@tcoratger tcoratger deleted the js-inspector branch August 14, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-rpc Related to the RPC implementation C-perf A change motivated by improving speed, memory usage or disk footprint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

perform js inspector on tracing task

2 participants