Skip to content
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.

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

Less alloc #625

Merged
merged 3 commits into from
Jun 30, 2020
Merged

Less alloc #625

merged 3 commits into from
Jun 30, 2020

Conversation

ucwong
Copy link
Member

@ucwong ucwong commented Jun 30, 2020

No description provided.

@codecov
Copy link

codecov bot commented Jun 30, 2020

Codecov Report

Merging #625 into master will decrease coverage by 0.05%.
The diff coverage is 75.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #625      +/-   ##
==========================================
- Coverage   46.44%   46.38%   -0.06%     
==========================================
  Files         398      398              
  Lines       50696    50707      +11     
==========================================
- Hits        23544    23519      -25     
- Misses      25371    25405      +34     
- Partials     1781     1783       +2     
Impacted Files Coverage Δ
cmd/utils/flags.go 2.15% <0.00%> (ø)
core/types/transaction.go 52.45% <14.28%> (-1.18%) ⬇️
core/tx_list.go 91.51% <100.00%> (ø)
core/tx_pool.go 76.60% <100.00%> (-0.81%) ⬇️
core/types/block.go 7.14% <100.00%> (+2.84%) ⬆️
crypto/crypto.go 63.47% <100.00%> (+0.64%) ⬆️
trie/committer.go 64.53% <100.00%> (ø)
trie/hasher.go 93.87% <100.00%> (ø)
consensus/cuckoo/sealer.go 0.00% <0.00%> (-6.76%) ⬇️
consensus/cuckoo/cuckoo.go 31.63% <0.00%> (-4.09%) ⬇️
... and 11 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 ade46bd...1466442. Read the comment docs.

@ucwong ucwong merged commit b232046 into master Jun 30, 2020
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.

None yet

2 participants