Skip to content

Conversation

ucwong
Copy link
Member

@ucwong ucwong commented Jul 24, 2023

No description provided.

@ucwong ucwong marked this pull request as ready for review July 24, 2023 07:09
@codecov
Copy link

codecov bot commented Jul 24, 2023

Codecov Report

Merging #1642 (5426542) into master (db9f229) will increase coverage by 0.04%.
The diff coverage is 45.55%.

❗ Current head 5426542 differs from pull request most recent head 20fbca5. Consider uploading reports for the commit 20fbca5 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1642      +/-   ##
==========================================
+ Coverage   52.53%   52.58%   +0.04%     
==========================================
  Files         480      481       +1     
  Lines       59986    60217     +231     
==========================================
+ Hits        31513    31663     +150     
- Misses      26095    26150      +55     
- Partials     2378     2404      +26     
Impacted Files Coverage Δ
cmd/cortex/main.go 21.73% <ø> (ø)
cmd/utils/flags.go 1.84% <0.00%> (-0.01%) ⬇️
core/state/iterator.go 0.00% <0.00%> (ø)
core/state/journal.go 72.54% <0.00%> (-2.97%) ⬇️
core/state/snapshot/conversion.go 0.00% <0.00%> (ø)
core/state/snapshot/generate.go 0.00% <0.00%> (ø)
core/state/snapshot/journal.go 0.00% <0.00%> (ø)
core/state/snapshot/snapshot.go 40.94% <0.00%> (-0.70%) ⬇️
core/types/state_account.go 0.00% <0.00%> (ø)
core/vm/gas_table.go 11.88% <ø> (+0.24%) ⬆️
... and 29 more

... and 10 files with indirect coverage changes

@ucwong ucwong merged commit c2fbe48 into master Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant