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

feat: wip gas metering#410

Closed
tynes wants to merge 1 commit intomainfrom
feat/gas-metering
Closed

feat: wip gas metering#410
tynes wants to merge 1 commit intomainfrom
feat/gas-metering

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented May 4, 2022

Description
Draft PR, WIP

Metadata

Fixes: ENG-2187

@codecov-commenter
Copy link

Codecov Report

Merging #410 (bc4a85f) into main (cc9f5c2) will increase coverage by 6.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #410      +/-   ##
==========================================
+ Coverage   50.06%   56.13%   +6.06%     
==========================================
  Files          51       59       +8     
  Lines        5151     6267    +1116     
==========================================
+ Hits         2579     3518     +939     
- Misses       2240     2352     +112     
- Partials      332      397      +65     
Impacted Files Coverage Δ
l2os/config.go 0.00% <0.00%> (ø)
opnode/service.go 0.00% <0.00%> (ø)
l2os/rollupclient/rollupclient.go 100.00% <0.00%> (ø)
opnode/l2/util.go 56.06% <0.00%> (ø)
bss/config.go 0.00% <0.00%> (ø)
bss/cmd/bss/main.go 0.00% <0.00%> (ø)
bss/flags/flags.go 100.00% <0.00%> (ø)
opnode/node/bundle_builder.go 100.00% <0.00%> (ø)
bss/batch_submitter.go 50.96% <0.00%> (ø)
bss/db/history_db.go 72.72% <0.00%> (ø)
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc9f5c2...bc4a85f. Read the comment docs.

@tynes
Copy link
Contributor Author

tynes commented May 11, 2022

This needs to be rebased on #421

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants