Skip to content

Conversation

@bricerisingalgorand
Copy link
Contributor

@bricerisingalgorand bricerisingalgorand commented Aug 6, 2021

GitHub Logo

IMPORTANT
This release requires a protocol upgrade.
This release contains a consensus protocol upgrade, which implements the following spec: https://github.com/algorandfoundation/specs/tree/abc54f79f9ad679d2d22f0fb9909fb005c16f8a1

Highlights

This release fixes issues with updating applications with ExtraProgramPages.

Changes

  1. TEAL

Additional Resources

shiqizng and others added 12 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.
@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2021

Codecov Report

Merging #2704 (8ce1e6c) into master (1ad773e) will increase coverage by 0.03%.
The diff coverage is 85.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2704      +/-   ##
==========================================
+ Coverage   46.96%   46.99%   +0.03%     
==========================================
  Files         348      349       +1     
  Lines       55715    55753      +38     
==========================================
+ Hits        26167    26203      +36     
+ Misses      26606    26605       -1     
- Partials     2942     2945       +3     
Impacted Files Coverage Δ
config/version.go 9.09% <ø> (ø)
ledger/apply/asset.go 19.89% <0.00%> (+0.21%) ⬆️
ledger/cow.go 89.28% <ø> (-1.04%) ⬇️
libgoal/libgoal.go 3.06% <0.00%> (-0.03%) ⬇️
protocol/consensus.go 0.00% <ø> (ø)
config/consensus.go 84.01% <100.00%> (+0.27%) ⬆️
data/basics/overflow.go 43.42% <100.00%> (+12.77%) ⬆️
data/transactions/transaction.go 32.09% <100.00%> (+1.26%) ⬆️
ledger/appcow.go 81.48% <100.00%> (+0.52%) ⬆️
ledger/apply/application.go 81.06% <100.00%> (+3.56%) ⬆️
... and 13 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 1ad773e...8ce1e6c. Read the comment docs.

@bricerisingalgorand bricerisingalgorand changed the base branch from master to rel/stable August 9, 2021 12:17
@bricerisingalgorand bricerisingalgorand marked this pull request as ready for review August 10, 2021 13:06
@algojohnlee algojohnlee merged commit 7b53f6f into rel/stable Aug 10, 2021
@algobarb algobarb deleted the hotfix/v2.9.1 branch August 30, 2021 20:37
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.

8 participants