Skip to content

CI: Fix some details around the hashKind -> HashKind change#4865

Merged
jannotti merged 2 commits intoalgorand:masterfrom
jannotti:hashkind-export
Dec 6, 2022
Merged

CI: Fix some details around the hashKind -> HashKind change#4865
jannotti merged 2 commits intoalgorand:masterfrom
jannotti:hashkind-export

Conversation

@jannotti
Copy link
Copy Markdown
Contributor

@jannotti jannotti commented Dec 6, 2022

hashKind was exported at the last moment, and some related code was not updated.

@jannotti jannotti changed the title Fix some details around the hashKind -> HashKind change CI: Fix some details around the hashKind -> HashKind change Dec 6, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 6, 2022

Codecov Report

Merging #4865 (8e28c24) into master (7b09c65) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #4865      +/-   ##
==========================================
- Coverage   53.34%   53.30%   -0.04%     
==========================================
  Files         423      423              
  Lines       53852    53852              
==========================================
- Hits        28725    28708      -17     
- Misses      22848    22862      +14     
- Partials     2279     2282       +3     
Impacted Files Coverage Δ
ledger/store/hashing.go 0.00% <ø> (ø)
ledger/store/hashkind_string.go 0.00% <0.00%> (ø)
ledger/roundlru.go 90.56% <0.00%> (-5.67%) ⬇️
crypto/merkletrie/trie.go 66.42% <0.00%> (-2.19%) ⬇️
network/wsPeer.go 67.06% <0.00%> (-1.91%) ⬇️
crypto/merkletrie/node.go 91.62% <0.00%> (-1.87%) ⬇️
catchup/service.go 69.08% <0.00%> (-0.25%) ⬇️
data/transactions/verify/txn.go 73.61% <0.00%> (ø)
ledger/testing/randomAccounts.go 56.61% <0.00%> (ø)
ledger/acctupdates.go 69.24% <0.00%> (+0.24%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants