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

fix: apply fix from release branch #815

Merged
merged 81 commits into from
Nov 29, 2022

Conversation

zemyblue
Copy link
Member

@zemyblue zemyblue commented Nov 29, 2022

Description

related: #813

apply fix from release/v0.46.x branch

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml

zemyblue and others added 30 commits October 27, 2022 18:27
Signed-off-by: zemyblue <[email protected]>

Signed-off-by: zemyblue <[email protected]>
* docs: fix submit-proposal tx command

* docs: add changelog
* Export pool

* Update CHANGELOG.md
…thority (Finschia#773)

* Guide users to use generate-only for certain cli commands

* Update CHANGELOG.md

* Update x/foundation/client/cli/tx.go
* feat: improve GetLastCompletedUpgrade

* rename

* use var block
…nterfaces (backport #12603) (#12638)

* feat: Move AppModule.BeginBlock and AppModule.EndBlock to extension interfaces (#12603)

Most modules right now have a no-op for AppModule.BeginBlock and AppModule.EndBlock. We should move these methods off the AppModule interface so we have less deadcode, and instead move them to extension interfaces.

1. I added `BeginBlockAppModule` and `EndBlockAppModule` interface.
2. Remove the dead-code from modules that do no implement them
3. Add type casting in the the module code to use the new interface

Closes: #12462

---

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] followed the guidelines for [building modules](https://github.com/cosmos/cosmos-sdk/blob/main/docs/building-modules)
- [ ] included the necessary unit and integration [tests](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#testing)
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

*All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.*

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed `!` in the type prefix if API or client breaking change
- [ ] confirmed all author checklist items have been addressed
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)

(cherry picked from commit b65f3fe)

* remove conflicts

* remove conflicts

* remove unused modules

Co-authored-by: Sishir Giri <[email protected]>
Co-authored-by: marbar3778 <[email protected]>
… balance for a given denom (backport #12674) (#12745)

* feat: Add convenience method for constructing key to access account's balance for a given denom (#12674)

This PR adds a convenience method for constructing the key necessary to query for the account's balance of a given denom.

I ran into this issue since we are using ABCI query now to perform balance requests because we are also requesting merkle proofs for the returned balance [here](https://github.com/celestiaorg/celestia-node/pull/911/files#diff-0ee31f5a7bd88e9f758e6bebdf3ee36365519e55a451098d9638c39afe5eac42R144).

It would be nice to have a definitive convenience method for constructing the key.

[Ref.](github.com/celestiaorg/celestia-node/pull/911)

(cherry picked from commit a1777a8)

* Update CHANGELOG.md

* fix conflict

Co-authored-by: rene <[email protected]>
Co-authored-by: Marko <[email protected]>
* bump tendermint version

* add changelog entry

* replace on jhump

* updates

* updates

* updates

Co-authored-by: Aleksandr Bezobchuk <[email protected]>
* prepare for release

* modify release notes
* fix(docs): typo in staking/state (#12834)

(cherry picked from commit fe89212)

# Conflicts:
#	x/staking/spec/01_state.md

* updates

Co-authored-by: Ari Rubinstein <[email protected]>
Co-authored-by: Julien Robert <[email protected]>
…11737) (#12818)

* fix: Use fixed length hex for pointer at FwdCapabilityKey (backport #11737)

* Update CHANGELOG.md

* add comments and unit tests
* feat: deterministic map iteration (#12781)

We should ensure that events emitted are in a deterministic order on any node.

Sorry, the previous [PR-12693](cosmos/cosmos-sdk#12693) was closed due to wrong operation.

Close: #12693

---

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [x] added `!` to the type prefix if API or client breaking change
- [x] targeted the correct branch (see [PR Targeting](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#pr-targeting))
- [x] provided a link to the relevant issue or specification
- [x] followed the guidelines for [building modules](https://github.com/cosmos/cosmos-sdk/blob/main/docs/building-modules)
- [x] included the necessary unit and integration [tests](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#testing)
- [x] added a changelog entry to `CHANGELOG.md`
- [x] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [x] updated the relevant documentation or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed

*All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.*

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed `!` in the type prefix if API or client breaking change
- [ ] confirmed all author checklist items have been addressed
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)

(cherry picked from commit 6ed11b8)

* resolve conflict (#12947)

* go mod tidy

* fix test

Co-authored-by: bruce-wayne2 <[email protected]>
Co-authored-by: Julien Robert <[email protected]>
…050)

* fix: missing return statement in BaseApp.Query (#13046)

Closes: #13040

---

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] followed the guidelines for [building modules](https://github.com/cosmos/cosmos-sdk/blob/main/docs/building-modules)
- [ ] included the necessary unit and integration [tests](https://github.com/cosmos/cosmos-sdk/blob/main/CONTRIBUTING.md#testing)
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

*All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.*

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed `!` in the type prefix if API or client breaking change
- [ ] confirmed all author checklist items have been addressed
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)

(cherry picked from commit c73171f)

* fix conflicts

Co-authored-by: Julien Robert <[email protected]>
* chore: v0.45.8 release changelog

* fix issue name
…kport #13107) (#13109)

* fix: call `SetIAVLCacheSize` with the configured value in simapp (#13107)

* Call `SetIAVLCacheSize` with the configured value in simapp.

* Update CHANGELOG.md

(cherry picked from commit ab33342)

* fix conflicts

Co-authored-by: yihuang <[email protected]>
… (backport #12742) (#13120)

* feat: Add a cli cmd to prune old states according to current settings (#12742)

* add PruningCmd and change PruneStores signature

* the mimimum default pruning interval is 10

Co-authored-by: Marko <[email protected]>
(cherry picked from commit d874ace)

* fix backport error and conflicts

Co-authored-by: adu-crypto <[email protected]>
Co-authored-by: adu <[email protected]>
ulbqb and others added 16 commits November 14, 2022 16:33
* Add libsodium option

* Add libsodium submodule

* Ignore tools/sodium

* Install and link libsodium

* Remove debug code

* Change to https submodule

* Update CHANGELOG

* Update ci tests
* Add field checks after x/token Modify

* Fix substitution logic in ModifyContract()

* Update CHANGELOG.md
* Add test.yml to diff target

* install libsodium

* Update CHANGELOG

* Update CHANGELOG

* Use `GITHUB_WORKSPACE`

* Fix test.yml

* Fix test.yml

* Fix test.yml

* Fix test.yml
…Finschia#784)

* Apply more strict field key name matching on x/token

* Lint

* Apply more strict field key name matching on x/collection

* Lint

* Update CHANGELOG.md

* Update msgs_test.go

Lint
* Update descriptions in CLI

* Update x/foundation README.md

* Update CHANGELOG.md
Signed-off-by: zemyblue <[email protected]>

Signed-off-by: zemyblue <[email protected]>
* chore: update ostracon v1.0.7

Signed-off-by: zemyblue <[email protected]>

* chore: update changelog

Signed-off-by: zemyblue <[email protected]>

Signed-off-by: zemyblue <[email protected]>
(update changelogs)

Signed-off-by: zemyblue <[email protected]>

Signed-off-by: zemyblue <[email protected]>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 18 committers have signed the CLA.

✅ zemyblue
✅ dudong2
✅ 0Tech
✅ ulbqb
❌ robert-zaremba
❌ adu-web3
❌ mergify[bot]
❌ julienrbrt
❌ fedekunze
❌ ebuchman
❌ yihuang
❌ tac0turtle
❌ faddat
❌ nddeluca
❌ SpicyLemon
❌ alexanderbez
❌ aaronc
❌ ValarDragon
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link

codecov bot commented Nov 29, 2022

Codecov Report

Merging #815 (d39d077) into main (451b622) will decrease coverage by 0.38%.
The diff coverage is 31.57%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #815      +/-   ##
==========================================
- Coverage   62.17%   61.78%   -0.39%     
==========================================
  Files         880      883       +3     
  Lines       99479   100381     +902     
==========================================
+ Hits        61852    62022     +170     
- Misses      34009    34724     +715     
- Partials     3618     3635      +17     
Impacted Files Coverage Δ
baseapp/grpcserver.go 1.72% <0.00%> (-0.04%) ⬇️
baseapp/msg_service_router.go 80.88% <ø> (ø)
baseapp/options.go 66.92% <0.00%> (-4.98%) ⬇️
client/grpc/node/query.pb.gw.go 0.00% <0.00%> (ø)
client/grpc/tmservice/service.go 68.75% <ø> (ø)
client/tx/legacy.go 62.50% <ø> (+1.52%) ⬆️
codec/amino_codec.go 100.00% <ø> (ø)
codec/proto_codec.go 74.33% <ø> (ø)
codec/types/any.go 54.16% <ø> (ø)
codec/unknownproto/unknown_fields.go 73.54% <ø> (ø)
... and 209 more

@zemyblue
Copy link
Member Author

In this PR, I want to ignore the code-cov error and lincense/cla CI error.

Copy link
Collaborator

@0Tech 0Tech left a comment

Choose a reason for hiding this comment

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

LGTM

@zemyblue zemyblue merged commit ffb28cd into Finschia:main Nov 29, 2022
@zemyblue zemyblue deleted the apply_v0.46.0_changes branch December 22, 2022 01:44
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.