Skip to content

Conversation

@onetechnical
Copy link
Contributor

@onetechnical onetechnical commented Aug 13, 2021

No description provided.

shiqizng and others added 14 commits August 6, 2021 15:43
Fix for updating an application with extra program pages
Make sure there is no overflow on TotalExtraAppPages calculation + test
updateApplication does not enforce individual program length but looks like it should. This PR enforces that.
Delete `Balances.PutWithCreatable()` and add new functions that notify the COW that a creatable was created or deleted. Coming up next is similar two functions for asset/application local state, needed by indexer.
@algobarb algobarb self-requested a review August 13, 2021 14:25
algobarb
algobarb previously approved these changes Aug 13, 2021
@onetechnical onetechnical changed the title relstable 2.9.1 remerge go-algorand v2.9.1-stable re-merge Aug 13, 2021
@codecov-commenter
Copy link

codecov-commenter commented Aug 13, 2021

Codecov Report

Merging #2740 (9b1a532) into master (6a63559) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2740      +/-   ##
==========================================
- Coverage   47.10%   47.09%   -0.01%     
==========================================
  Files         349      349              
  Lines       56321    56321              
==========================================
- Hits        26529    26525       -4     
  Misses      26819    26819              
- Partials     2973     2977       +4     
Impacted Files Coverage Δ
config/version.go 9.09% <ø> (ø)
ledger/blockqueue.go 81.03% <0.00%> (-4.03%) ⬇️
crypto/merkletrie/trie.go 66.42% <0.00%> (-2.19%) ⬇️
crypto/merkletrie/node.go 91.62% <0.00%> (-1.87%) ⬇️
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
network/requestTracker.go 70.25% <0.00%> (-0.87%) ⬇️
data/transactions/verify/txn.go 45.08% <0.00%> (ø)
network/wsNetwork.go 60.92% <0.00%> (+0.18%) ⬆️
network/wsPeer.go 74.93% <0.00%> (+0.55%) ⬆️
ledger/acctupdates.go 62.63% <0.00%> (+0.58%) ⬆️
... and 1 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 6a63559...9b1a532. Read the comment docs.

@algojohnlee algojohnlee merged commit 045ffff into master Aug 13, 2021
@algojohnlee algojohnlee deleted the relstable-2.9.1-remerge branch August 13, 2021 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants