Skip to content

Confirm make generate fails with stale changes#4866

Closed
michaeldiamant wants to merge 1 commit into
algorand:masterfrom
michaeldiamant:make_generate_test
Closed

Confirm make generate fails with stale changes#4866
michaeldiamant wants to merge 1 commit into
algorand:masterfrom
michaeldiamant:make_generate_test

Conversation

@michaeldiamant
Copy link
Copy Markdown
Contributor

Confirms #4865 fails with stale generated source changes. Will close once codegen verification fails.

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 6, 2022

Codecov Report

Merging #4866 (3b95dd3) into master (7b09c65) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4866   +/-   ##
=======================================
  Coverage   53.34%   53.34%           
=======================================
  Files         423      423           
  Lines       53852    53852           
=======================================
+ Hits        28725    28730    +5     
+ Misses      22848    22844    -4     
+ Partials     2279     2278    -1     
Impacted Files Coverage Δ
ledger/roundlru.go 90.56% <0.00%> (-5.67%) ⬇️
crypto/merkletrie/trie.go 66.42% <0.00%> (-2.19%) ⬇️
crypto/merkletrie/node.go 91.62% <0.00%> (-1.87%) ⬇️
network/wsNetwork.go 64.74% <0.00%> (-0.18%) ⬇️
ledger/acctupdates.go 69.24% <0.00%> (+0.24%) ⬆️
network/wsPeer.go 69.45% <0.00%> (+0.47%) ⬆️
ledger/acctonline.go 78.81% <0.00%> (+0.51%) ⬆️
ledger/testing/randomAccounts.go 57.23% <0.00%> (+0.61%) ⬆️
ledger/tracker.go 75.94% <0.00%> (+1.68%) ⬆️
ledger/blockqueue.go 88.50% <0.00%> (+2.87%) ⬆️

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

@michaeldiamant
Copy link
Copy Markdown
Contributor Author

@michaeldiamant michaeldiamant deleted the make_generate_test branch December 6, 2022 17:22
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