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

build(deps): bump github.com/tendermint/tendermint from v0.34.21 to v0.34.22 #83

Merged
merged 14 commits into from
Nov 24, 2022

Conversation

RiccardoM
Copy link
Contributor

@RiccardoM RiccardoM commented Nov 23, 2022

Description

This PR bumps Tendermint to v0.34.22 removing the need to replace the default dependency thanks to the newly added support for https websocket URLs directlty inside Tendermint Core.

Checklist

  • Targeted PR against correct branch.
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote unit tests.
  • Re-reviewed Files changed in the Github PR explorer.

@RiccardoM RiccardoM changed the base branch from cosmos/v0.44.x to cosmos/v0.45.x November 23, 2022 17:09
RiccardoM and others added 8 commits November 23, 2022 12:10
This PR updates the `PostgreSQL` database to use the sqlx driver instead of `sql` to provide more functionalities while maintaining backward compatibility

- [x] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Wrote unit tests.
- [x] Re-reviewed `Files changed` in the Github PR explorer.
This PR updates the linting process to use the `golangci-lint` CLI. This should guarantee major forward compatibility as the golangci-lint version is now tracked within the codebase along with other dependencies

Depends-On: #79

- [x] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Wrote unit tests.
- [x] Re-reviewed `Files changed` in the Github PR explorer.
…ing (#74)

Fixes [BDU-542](https://forbole.atlassian.net/browse/BDU-542)

- [x] Targeted PR against correct branch.
- [x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Wrote unit tests.
- [x] Re-reviewed `Files changed` in the Github PR explorer.
@RiccardoM RiccardoM force-pushed the riccardom/bump-tendermint branch from e72dc4e to 6b88fb0 Compare November 23, 2022 17:12
@RiccardoM RiccardoM requested review from huichiaotsou and MonikaCat and removed request for huichiaotsou November 23, 2022 17:26
@RiccardoM RiccardoM added the automerge Automatically merge PR once all prerequisites pass label Nov 23, 2022
Copy link
Contributor

@MonikaCat MonikaCat left a comment

Choose a reason for hiding this comment

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

tested, LGTM

Co-authored-by: Magic Cat <37407870+MonikaCat@users.noreply.github.com>
@mergify mergify bot merged commit d4e57e9 into cosmos/v0.45.x Nov 24, 2022
@mergify mergify bot deleted the riccardom/bump-tendermint branch November 24, 2022 12:54
RiccardoM added a commit that referenced this pull request Nov 24, 2022
…0.34.22 (#83)

This PR bumps Tendermint to `v0.34.22` removing the need to replace the default dependency thanks to the newly added support for `https` websocket URLs directlty inside Tendermint Core.

- [x] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Wrote unit tests.
- [x] Re-reviewed `Files changed` in the Github PR explorer.
MonikaCat pushed a commit that referenced this pull request Jan 19, 2024
…0.34.22 (#83)

This PR bumps Tendermint to `v0.34.22` removing the need to replace the default dependency thanks to the newly added support for `https` websocket URLs directlty inside Tendermint Core.
- [x] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
- [ ] Wrote unit tests.
- [x] Re-reviewed `Files changed` in the Github PR explorer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatically merge PR once all prerequisites pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants