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

feat: bump cosmos-sdk to v0.50.6 #114

Merged
merged 2 commits into from
Apr 23, 2024
Merged

feat: bump cosmos-sdk to v0.50.6 #114

merged 2 commits into from
Apr 23, 2024

Conversation

JulianToledano
Copy link
Contributor

No description provided.

Copy link

coderabbitai bot commented Apr 23, 2024

Walkthrough

Version v0.50.6 of cosmos/rosetta includes improvements related to updating cosmos-sdk to v0.50.6. This update likely involves enhancements or bug fixes that align with the ongoing development and maintenance of the software, ensuring compatibility and functionality within the Cosmos ecosystem.

Changes

File(s) Change Summary
CHANGELOG.md Updated to reflect improvements in version v0.50.6 related to updating cosmos-sdk to v0.50.6.

Possibly related issues

  • [Feature]: Update cosmos-sdk to v0.50 release #57: The PR improvement in cosmos/rosetta to v0.50.6 aligns with the request in this issue to update cosmos-sdk to version v0.50, indicating a potential alignment in the dependency or compatibility adjustments.

Recent Review Details

Configuration used: CodeRabbit UI
Review profile: CHILL

Commits Files that changed from the base of the PR and between d5ee830 and f053ccc.
Files selected for processing (1)
  • CHANGELOG.md (1 hunks)
Additional Context Used
LanguageTool (48)
CHANGELOG.md (48)

Near line 39: Unpaired symbol: ‘[’ seems to be missing
Context: ... Changelog ## [Unreleased] ## [v0.50.6](https://github.com/cosmos/rosetta/relea...


Near line 43: Unpaired symbol: ‘[’ seems to be missing
Context: ....6) 2024-04-23 ### Improvements * [114](https://github.com/cosmos/rosetta/pull/...


Near line 43: Possible spelling mistake found.
Context: ...hub.com//pull/114) Update cosmos-sdk to v0.50.6 ## [v0.50.5](https://github...


Near line 45: Unpaired symbol: ‘[’ seems to be missing
Context: ...pdate cosmos-sdk to v0.50.6 ## [v0.50.5](https://github.com/cosmos/rosetta/relea...


Near line 49: Unpaired symbol: ‘[’ seems to be missing
Context: ...0.50.5) 2024-03-18 ### Bug Fixes * [99](https://github.com/cosmos/rosetta/pull/...


Near line 49: Possible spelling mistake found.
Context: ...tta/pull/99) Rosetta now can connect to cometBFT over HTTPS. ### Improvements * [93](h...


Near line 53: Unpaired symbol: ‘[’ seems to be missing
Context: ...BFT over HTTPS. ### Improvements * [93](https://github.com/cosmos/rosetta/pull/...


Near line 53: Add a space between sentences.
Context: ...mos/rosetta/pull/93) Removes the use of LegacyMsg.GetSigners() in favor of codec.GetMsgV1Signers....


Near line 53: Add a space between sentences.
Context: ...of LegacyMsg.GetSigners() in favor of codec.GetMsgV1Signers. * [#101](https://github.com/cosmos/ro...


Near line 54: Unpaired symbol: ‘[’ seems to be missing
Context: ...avor of codec.GetMsgV1Signers. * [#101](https://github.com/cosmos/rosetta/pull/...


Near line 54: Possible spelling mistake found.
Context: ....com//pull/101) Update to cosmos-sdk v0.50.5 ## [v0.50.4](https://github.co...


Near line 56: Unpaired symbol: ‘[’ seems to be missing
Context: ...pdate to cosmos-sdk v0.50.5 ## [v0.50.4](https://github.com/cosmos/rosetta/relea...


Near line 60: Unpaired symbol: ‘[’ seems to be missing
Context: ....4) 2024-02-26 ### Improvements * [#88](https://github.com/cosmos/rosetta/pull/...


Near line 60: Possible spelling mistake found.
Context: ...b.com//pull/88) Update to cosmos-sdk v0.50.4 ## [v0.50.3+1](https://github....


Near line 62: Unpaired symbol: ‘[’ seems to be missing
Context: ...ate to cosmos-sdk v0.50.4 ## [v0.50.3+1](https://github.com/cosmos/rosetta/relea...


Near line 68: Unpaired symbol: ‘[’ seems to be missing
Context: ...0.3+1 instead. ### Improvements * [#73](https://github.com/cosmos/rosetta/pull/...


Near line 68: Possible spelling mistake found.
Context: ...b.com//pull/73) Update to cosmos-sdk v0.50.3 * [#70](https://github.com/cosm...


Near line 69: Unpaired symbol: ‘[’ seems to be missing
Context: .../73) Update to cosmos-sdk v0.50.3 * [#70](https://github.com/cosmos/rosetta/pull/...


Near line 69: Possible spelling mistake found.
Context: ...smos/rosetta/pull/70) Coinbase accurate dockerfile. ### Bug Fixes * [#82](https://github...


Near line 73: Unpaired symbol: ‘[’ seems to be missing
Context: ...urate dockerfile. ### Bug Fixes * [#82](https://github.com/cosmos/rosetta/pull/...


Near line 74: Unpaired symbol: ‘[’ seems to be missing
Context: ...mossdk.io/core dependencies. ## [v0.50.2](https://github.com/cosmos/rosetta/relea...


Near line 78: Unpaired symbol: ‘[’ seems to be missing
Context: ....2) 2023-12-12 ### Improvements * [#58](https://github.com/cosmos/rosetta/pull/...


Near line 78: Possible spelling mistake found.
Context: ...ub.com//pull/58) Upgraded cosmos-sdk version and removed tip handling. * [#3...


Near line 79: Unpaired symbol: ‘[’ seems to be missing
Context: ...version and removed tip handling. * [#37](https://github.com/cosmos/rosetta/pull/...


Near line 79: Possible spelling mistake found.
Context: ...ps://github.com//pull/37) Dockerization of Rosetta. * [#29](https://github.com/...


Near line 80: Unpaired symbol: ‘[’ seems to be missing
Context: ...ull/37) Dockerization of Rosetta. * [#29](https://github.com/cosmos/rosetta/pull/...


Near line 84: Possible spelling mistake found.
Context: ...rror handling. ## v0.47.x * Migrated rosetta from cosmos-sdk repository to the stand...


Near line 84: Possible spelling mistake found.
Context: .... ## v0.47.x * Migrated rosetta from cosmos-sdk repository to the standalone [repo](htt...


Near line 88: Unpaired symbol: ‘[’ seems to be missing
Context: ...s/rosetta). ### Improvements * [#14272](https://github.com/cosmos/cosmos-sdk/pu...


Near line 88: Possible spelling mistake found.
Context: ...b.com/cosmos/cosmos-sdk/pull/14272) Use coinbase/rosetta-sdk-go/types packages instead ...


Near line 88: Possible spelling mistake found.
Context: ...setta-sdk-go/types` packages instead of comsos fork. ### Bug Fixes * [#14285](https:...


Near line 92: Unpaired symbol: ‘[’ seems to be missing
Context: ...f comsos fork. ### Bug Fixes * [#14285](https://github.com/cosmos/cosmos-sdk/pu...


Near line 92: Possible spelling mistake found.
Context: ....com/cosmos/cosmos-sdk/pull/14285) Sets tendermint errors status codes to 500 ## v0.2.0 2...


Near line 98: Unpaired symbol: ‘[’ seems to be missing
Context: ... 2022-12-07 ### Improvements * [#14118](https://github.com/cosmos/cosmos-sdk/pu...


Near line 98: Possible spelling mistake found.
Context: ...com/cosmos/cosmos-sdk/pull/14118) Allow rosetta to be installed as a standalone applica...


Near line 99: Unpaired symbol: ‘[’ seems to be missing
Context: ...d as a standalone application. * [#14061](https://github.com/cosmos/cosmos-sdk/pu...


Near line 99: Possible spelling mistake found.
Context: ....com/cosmos/cosmos-sdk/pull/14061) Adds openapi specification. * [#13832](https://githu...


Near line 100: Unpaired symbol: ‘[’ seems to be missing
Context: ...1) Adds openapi specification. * [#13832](https://github.com/cosmos/cosmos-sdk/pu...


Near line 100: Possible spelling mistake found.
Context: ...mos-sdk/pull/13832) Correctly populates rosetta's /network/status endpoint response. ...


Near line 100: Possible spelling mistake found.
Context: ...atus` endpoint response. Rosetta's data api is divided into its own go files (accou...


Near line 100: Possible spelling mistake found.
Context: ... into its own go files (account, block, mempool, network). ### Bug Fixes * [#13832](h...


Near line 104: Unpaired symbol: ‘[’ seems to be missing
Context: ...ool, network). ### Bug Fixes * [#13832](https://github.com/cosmos/cosmos-sdk/pu...


Near line 104: Possible spelling mistake found.
Context: ....com/cosmos/cosmos-sdk/pull/13832) Wrap tendermint RPC errors to rosetta errors. ## v0.1....


Near line 104: Possible spelling mistake found.
Context: ...ll/13832) Wrap tendermint RPC errors to rosetta errors. ## v0.1.0 2022-11-04 **From `...


Near line 108: Possible spelling mistake found.
Context: ... From v0.1.0 the minimum version of Tendermint is v0.37+, due event type changes. ...


Near line 108: Possible missing preposition found.
Context: ... version of Tendermint is v0.37+, due event type changes.** ### Improvements * [#...


Near line 112: Unpaired symbol: ‘[’ seems to be missing
Context: ... changes.** ### Improvements * [#13583](https://github.com/cosmos/cosmos-sdk/pu...


Near line 112: Possible spelling mistake found.
Context: ...m/cosmos/cosmos-sdk/pull/13583) Extract rosetta to its own go.mod.

Additional comments not posted (1)
CHANGELOG.md (1)

39-44: The changelog update for version v0.50.6 is correctly formatted and accurately describes the changes made. Good job maintaining clarity and consistency in the changelog entries.


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

Note: Auto-reply has been disabled for this repository by the repository owner. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Out of diff range and nitpick comments (1)
CHANGELOG.md (1)

39-44: Ensure the entry for PR 114 is descriptive.

The changelog entry for PR 114 currently only includes the PR link. It's best practice to provide a brief description of the changes or improvements made in the PR to help users understand the context without having to navigate to the PR page.

@JulianToledano JulianToledano merged commit 92c741d into main Apr 23, 2024
10 checks passed
@JulianToledano JulianToledano deleted the feat/update-v0.50.6 branch April 23, 2024 11:16
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.

2 participants