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.

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

Release v0.39.2 #7574

Closed
wants to merge 26 commits into from
Closed

Conversation

alessio
Copy link
Contributor

@alessio alessio commented Oct 16, 2020

Cosmos SDK v0.39.2 Release.

Release milestone.


Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer
  • Review Codecov Report in the comment section below once CI passes

Alessio Treglia and others added 16 commits August 18, 2020 17:20
Replace old actions that don't build anymore.
Backported from #7084

Co-authored-by: Alessio Treglia <[email protected]>
Fix the command line `keys add` interactive mode
that fails when asked to generate new mnemonic.

Closes: #7048
From:
- #7049
- #7183
Thanks: @puneet2019 @PaddyMc for the patches.
Added a maximum number of iterations (100) to ApproxRoot
(and ApproxSqrt) to serve as a hard limit on the number of
iterations that the algorithm performs. If the answer converges
before the maximum iterations, it returns immediately. Otherwise,
if the answer never converges enough to satisfy the primary loop
condition, the looping stops after a hundred iterations.

Closes: #7038

Co-authored-by: Miguel Dingli <[email protected]>
The release contains a fix for dvsekhvalnov/jose2go#26,
which is needed for the file keyring backend to work with go 1.15.

From: #7397
Update vuepress/config & package.json to match master.

Touch package-lock.json.
Update tendermint to 0.33.8, see [1] for more information.

[1] golang/go#40618
@yun-yeo
Copy link
Contributor

yun-yeo commented Oct 21, 2020

Can you add #7589 PR to backports?

@alessio
Copy link
Contributor Author

alessio commented Oct 21, 2020

Can you add #7589 PR to backports?

Happy to open a PR 👍

@yun-yeo
Copy link
Contributor

yun-yeo commented Oct 22, 2020

Can you add #7589 PR to backports?

Happy to open a PR 👍

Is it correct way?
#7624

From: #7589
Closes: #7588
Thanks: @YunSuk-Yeo 

Co-authored-by: Alessio Treglia <[email protected]>
Copy link
Contributor

@ethanfrey ethanfrey left a comment

Choose a reason for hiding this comment

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

Looks good, I approved all those PRs in here.

One minor comment. We typically squash all PRs into one commit when merging, but
c3cc14c ... 3101928 all come from one pr to allow emoji token names. Maybe we squash them before merging into release?

Alessio Treglia added 2 commits November 10, 2020 19:06
The output of the version --long command now shows
the list of build dependencies.

From: #7848
Closes: #7835
@alessio
Copy link
Contributor Author

alessio commented Nov 13, 2020

Alright, release notes seem to be final. I'm taggint v0.39.2-rc1. Now we need testing

@alessio alessio marked this pull request as ready for review November 13, 2020 19:06
CHANGELOG.md Show resolved Hide resolved
@alessio
Copy link
Contributor Author

alessio commented Nov 16, 2020

ACK received from Commerc.io: commercionetwork/commercionetwork#177

We still await more feedback.

@tac0turtle
Copy link
Member

ci fix is here: #7969

@fedekunze
Copy link
Collaborator

also relevant #7971

@alessio
Copy link
Contributor Author

alessio commented Nov 18, 2020

ACK received from Terra, see terra-money/classic-core#430

fedekunze and others added 2 commits November 18, 2020 20:18
* launchpad: bump tendermint to v0.33.9

* update release notes

Co-authored-by: Alessio Treglia <[email protected]>
@alessio alessio changed the title Launchpad/backports Release v0.39.2 Nov 19, 2020
* launchpad: backport 7970

* cherry pick 7628

* fix tests

* Update CHANGELOG.md

* revert tiny portion of #7970 (#7984)

Testable examples were accidentally converted into tests.

* adapt test to launchpad

* fix golangci-lint warnings

Co-authored-by: Emmanuel T Odeke <[email protected]>
Co-authored-by: Alessio Treglia <[email protected]>
@lgtm-com
Copy link

lgtm-com bot commented Nov 20, 2020

This pull request fixes 1 alert when merging 5be42d9 into 6a624c3 - view on LGTM.com

fixed alerts:

  • 1 for Incorrect conversion between integer types

@@ -1,13 +1,5 @@
# CODEOWNERS: https://help.github.com/articles/about-codeowners/

# Primary repo maintainers
Copy link
Contributor

Choose a reason for hiding this comment

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

Any particular reason for these changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changes committed as per Launchpad's release policy. The Policy states that PRs to the stable release branch must be approved by the Stable Release Managers.
Changes were approved offline by @okwme.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok this is just for launchpad?

Also, why is the base branch launchpad/release/v0.39.2? It should be release/v0.39.x or something similar.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok this is just for launchpad?

Currently yes, correct - although IIRC people liked the concept of having a small dedicated group of people maintaining each stable release branch. Yet yes of course, this applies to Launchpad only.

Also, why is the base branch launchpad/release/v0.39.2?

There should be ACLs enforced on branches that start with launchpad/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Anyways, the release branch will be removed once the final tag is pushed

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure I understand but ok!

…8001)

types/Coin: compile and reuse Regexps to reduce massive RAM+CPU burn (#7989)

From: #7989
Closes: #7986

Co-authored-by: Federico Kunze <[email protected]>
Co-authored-by: Emmanuel T Odeke <[email protected]>
@lgtm-com
Copy link

lgtm-com bot commented Nov 24, 2020

This pull request fixes 1 alert when merging 2727c14 into 6a624c3 - view on LGTM.com

fixed alerts:

  • 1 for Incorrect conversion between integer types

@alessio
Copy link
Contributor Author

alessio commented Nov 24, 2020

We're ready for liftoff:

  • Tests are green
  • Client testing successful, no regressions were reported.
  • All Stable Release Managers have signed off.

Hence tagging (git tag -a) and closing.

Thank you all very much indeed for making Cosmos SDK better.

@alessio alessio closed this Nov 24, 2020
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.

10 participants