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

Merge v1.8.2 changes back to main #801

Merged
merged 37 commits into from
Apr 25, 2022
Merged

Conversation

dwedul-figure
Copy link
Contributor

Description

This merges the v1.8.2 changes back into main.


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.
  • 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

egaxhaj and others added 30 commits January 28, 2022 14:44
* add java distribution

* update changelog
…#636)

* add java distribution (#625)

* add java distribution

* update changelog

* Update gov module deposit requirements and add msg fee (#633)

* add deposit and msg fee updates for gov in upgrade

* changelog

* Allow attributes to be associated with scopes. (#634)

* [631]: Update NewAttribute to take in a string instead of an AccAddress.

* [631]: Update Attribute.ValidateBasic() to allow the address to be a scope metadata address.

* [631]: Update NewMsgAddAttributeRequest to take in a string for the account instead of an AccAddress in order to allow for scope metadata addresses.

* [631]: Update NewMsgUpdateAttributeRequest to take in a string for the account instead of an AccAddress in order to allow for scope metadata addresses.

* [631]: Update the attribute keeper methods to treat the Address as a string instead of AccAddress to allow it to be a scope metadata address.

* [631]: Update the attribute keys methods to not rely on the address being an account address.

* [631]: Lint fixes.

* [631]: Lint fixes in main that make lint is complaining about.

* [631]: Add changelog entry.

* [631]: Add some stuff that was missed like the delete messages and some extra context on returned errors.

* Fix message breaking unit test

Co-authored-by: Derek Adams <dadams@figure.com>
Co-authored-by: Derek Adams <derek.adams@sitewhere.com>
Co-authored-by: Ira Miller <72319+iramiller@users.noreply.github.com>

Co-authored-by: Ergels Gaxhaj <87285445+egaxhaj-figure@users.noreply.github.com>
Co-authored-by: Daniel Wedul <72031080+dwedul-figure@users.noreply.github.com>
Co-authored-by: Derek Adams <dadams@figure.com>
Co-authored-by: Derek Adams <derek.adams@sitewhere.com>
* Add msgfees to store added upgrades

* Add changelog entry
* 614 proto binding support with buf build (#630)

* initial attemp at using buf.build

* buf.build stuff

* buf workflow

* trigger on PR for testing

* fix breaking for third_party

* use existing workflow

* updated proto steps

* changelog entry

* fix bug with proto-all

* update proto script and protos

* update proto-all task

* fix typo

* fix bug with proto binding release version & combine workflows

* add protoBinding build step to release.yml workflow

* missed in git add on last commit

* fix lint errors

* fix buf braking
* fix release version reference

* update changelog
* Bump cosmwasm from v1.0.0-beta5 to v1.0.0-beta6, Add change log entry

* Add dockerfile cosmoswasm v1.0.0-beta5 direct download and validation

* Revert dockerfile changes
* Add iterator as a feature to wasm
* remove v from artifact version

* update changelog
* Change go.mod to use wasmd fork to support old address length

* Linked issue
* adding upgrade handler indigo.
Upgrade golangci-lint and fix lint errors
#705)

* backport: In release 1.8(not yet in mainnet), floor gas price should be checked for all Tx's ( currently only being checked for Tx which have additional Msg Fee),closes: #684
backport: fix owner array initialization size when creating scope mutation wasm msg
* simulation tests are running

* fixed record issue

* Add smart contract to github actions testing

* fixed determinism test

* cleanup

* make format

* cleanup

* make format

* lint changes

* more lint fixes

* fix tests

* reordered sims tests

* working on import tests

* fixed tests

* cleanup

* add changelog

* fix test name

* make format

* update linter

* lint error

* fix append errors

* lint fixes

* update lint version

* print line numbers

* use verbose output

* use work around for linter

* reverted name change

* more clenaup

* nolint comments

* more nolint comments

* updated linter for kErr

* update golint version

* nolint for gosec

* make format

* remove unused consts

* nolint cleanups

* make format

* update action version

* revert action change
* Skip max gas fees if tx contains MsgSubmitProposal msg (#711)

* Add jasmine upgrade handler
* pre rc10 (new handler, gas fee 15/blk, ante reorder)

Co-authored-by: arnabmitra <arnabmitra.mitra@gmail.com>
* Add script to download latest provwasm smart contract and require it in tests

* cleanup and changelog
* Update the github workflows with version changes that were made to master.

* Bump ibc-go to v2.2.0 (from v2.0.2). This change has already been made in master, just bringing it in here too.
* updates required for cosmos sdk upgrade

* improve changelog

* improve issue description
* enable adr 038 state listening

* fix goimports lint error

* panic on error

* v1.0.0-beta4 not yet released

* copy missing mk file

* update changelog

Co-authored-by: Ergels Gaxhaj <egaxhaj@figure.com>
# Conflicts:
#	.github/workflows/proto.yml
#	.github/workflows/release.yml
#	.github/workflows/sims.yml
#	.github/workflows/test.yml
#	app/sim_contracts/tutorial.wasm
#	app/sim_test.go
#	docker/blockchain/Dockerfile
#	go.mod
#	go.sum
#	internal/antewrapper/msg_fees_decorator_test.go
#	internal/antewrapper/tx_gas_limit_decorator.go
#	proto/provenance/msgfees/v1/msgfees.proto
#	scripts/proto-update-deps.sh
#	sims.mk
#	x/marker/simulation/operations.go
#	x/msgfees/types/msgfees.pb.go
# Conflicts:
#	.github/workflows/proto.yml
#	.github/workflows/release.yml
#	.github/workflows/sims.yml
#	.github/workflows/test.yml
#	app/sim_contracts/tutorial.wasm
#	app/sim_test.go
#	docker/blockchain/Dockerfile
#	go.mod
#	go.sum
#	internal/antewrapper/msg_fees_decorator_test.go
#	internal/antewrapper/tx_gas_limit_decorator.go
#	proto/provenance/msgfees/v1/msgfees.proto
#	scripts/proto-update-deps.sh
#	sims.mk
#	x/marker/simulation/operations.go
#	x/msgfees/types/msgfees.pb.go
@dwedul-figure dwedul-figure marked this pull request as ready for review April 25, 2022 18:54
@dwedul-figure dwedul-figure merged commit 7720099 into main Apr 25, 2022
@dwedul-figure dwedul-figure deleted the dwedul/v1.8.2-back-to-main branch April 25, 2022 19:03
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