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.

Sign up for GitHub

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

Jump to bottom

FOR REVIEW ONLY - Release v1.24.0 #703

Merged
merged 9 commits into from
Dec 5, 2022
Merged

FOR REVIEW ONLY - Release v1.24.0 #703

merged 9 commits into from
Dec 5, 2022

Conversation

algolucky
Copy link
Contributor

What's Changed

Bugfixes

Enhancements

New Contributors

Full Changelog: v1.23.2...v1.24.0

algolucky and others added 9 commits November 10, 2022 11:55
@algolucky
Merge branch 'release/v1.23.2' into develop
0720cd4
@algochoi @ahangsu
Tests: Migrate v1 algod dependencies to v2 in cucumber tests (#693)
a59c152 
* Remove algod v1 steps

* Revert deleted tags and add ledger supply check

* Fix ledger supply tests

* Remove unused steps

* Update .test-env

Co-authored-by: algochoi <86622919+algochoi@users.noreply.github.com>

Co-authored-by: Hang Su <87964331+ahangsu@users.noreply.github.com>
Co-authored-by: Hang Su <hang.su@algorand.com>
@michaeldiamant
Add KV counts to NodeStatusResponse (#696)
acb33b2
@nullun @algochoi
Fix createMultisigTransaction name in comments (#694)
3ad0913 
Co-authored-by: algochoi <86622919+algochoi@users.noreply.github.com>
@ahangsu
Enhancement: allowing zero-length static array (#698)
9225675
@ahangsu
Bug-Fix: encode ABI string with non-ASCII characters (#700)
4f0c9de
@michaeldiamant
ABI: Refactor ABI encoding test to round-trip (#701)
2cd2143
@algolucky
bump version and add to changelog
1c92414
@algolucky
update README.md for new version
9888f42
@algolucky algolucky self-assigned this Dec 5, 2022
michaeldiamant
michaeldiamant approved these changes Dec 5, 2022
View reviewed changes
Copy link
Contributor

@michaeldiamant michaeldiamant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commits, version bump, and README checksum updates look as intended.

@algolucky algolucky marked this pull request as ready for review December 5, 2022 19:03
@algolucky algolucky merged commit 607515e into master Dec 5, 2022
@algolucky algolucky deleted the release/v1.24.0 branch December 5, 2022 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaeldiamant michaeldiamant michaeldiamant approved these changes

@algochoi algochoi algochoi approved these changes

@ahangsu ahangsu ahangsu approved these changes

@jasonpaulos jasonpaulos Awaiting requested review from jasonpaulos

@excalq excalq Awaiting requested review from excalq excalq was automatically assigned from algorand/devops

@algobarb algobarb Awaiting requested review from algobarb algobarb was automatically assigned from algorand/devops

@algojack algojack Awaiting requested review from algojack algojack was automatically assigned from algorand/devops

Assignees

@algolucky algolucky

Labels
Skip-Release-Notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@algolucky @michaeldiamant @algochoi @ahangsu @nullun