chore: typos fix by misspell-fixer

[github-prbot]

This PR fixes typos in the codebase.
Please, review it and merge if everything is fine.
If there is proto changes, please, run make proto-gen and commit changes.

Summary by CodeRabbit

• Documentation

  • Improved clarity and fixed typos in various documentation files and comments.
  • Updated descriptions to reflect deprecation of certain fields in governance parameters.
  • Emphasized consistency in metadata storage descriptions for client applications.

• Bug Fixes

  • Corrected spelling errors in variable names and comments across multiple modules.

• Refactor

  • Deprecated fields in governance parameters to streamline the configuration process.

• Style

  • Standardized spelling for terms like "initializing" and "occurred" in comments and documentation.

[coderabbitai]

Walkthrough

Walkthrough

The changes across various files are primarily focused on correcting typographical errors and improving the clarity of comments and documentation. Key updates include deprecating fields in the Cosmos SDK's governance module and refining comments for consistency and accuracy. No changes to the logic or functionality of the code are indicated, suggesting a maintenance and documentation improvement effort.

Changes

File(s)	Summary
CHANGELOG.md, docs/architecture/adr-037-gov-split-vote.md, docs/architecture/adr-050-sign-mode-textual.md, docs/spec/fee_distribution/f1_fee_distr.tex, x/auth/vesting/README.md, x/authz/README.md, x/gov/README.md, x/staking/README.md	Corrected typographical errors in documentation and comments.
api/cosmos/gov/v1/genesis.pulsar.go, api/cosmos/gov/v1/query.pulsar.go, proto/cosmos/gov/v1/genesis.proto, proto/cosmos/gov/v1/query.proto	Updated comments and deprecated fields related to deposit, voting, and tally parameters.
api/cosmos/staking/v1beta1/staking.pulsar.go, proto/cosmos/staking/v1beta1/staking.proto	Fixed a typo from "occured" to "occurred" in comments.
docs/build/building-modules/01-module-manager.md	Corrected "Contructor" to "Constructor" in function description.
tests/e2e/auth/suite.go	Renamed variable from "intialCoins" to "initialCoins".
x/gov/types/config.go, x/group/internal/orm/sequence_property_test.go	Corrected "intialising" to "initialising" in comments.
x/nft/simulation/operations.go	Changed variable name from recieverStr to receiverStr.

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 X ?

---

Tips

Chat with CodeRabbit Bot (@coderabbitai)

• If you reply to a review comment from CodeRabbit, the bot will automatically respond.
• To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
• Note: Review comments are made on code diffs or files, not on the PR overview.
• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

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.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• The JSON schema for the configuration file is available here.
• 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/coderabbit-overrides.v2.json

[coderabbitai]

Review Status

Actionable comments generated: 0

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between 8cfdabb and 530163a.

Files ignored due to filter (3)

• x/gov/types/v1/genesis.pb.go
• x/gov/types/v1/query.pb.go
• x/staking/types/staking.pb.go

Files selected for processing (19)

• CHANGELOG.md (3 hunks)
• api/cosmos/gov/v1/genesis.pulsar.go (1 hunks)
• api/cosmos/gov/v1/query.pulsar.go (1 hunks)
• api/cosmos/staking/v1beta1/staking.pulsar.go (1 hunks)
• docs/architecture/adr-037-gov-split-vote.md (1 hunks)
• docs/architecture/adr-050-sign-mode-textual.md (1 hunks)
• docs/build/building-modules/01-module-manager.md (1 hunks)
• docs/spec/fee_distribution/f1_fee_distr.tex (1 hunks)
• proto/cosmos/gov/v1/genesis.proto (1 hunks)
• proto/cosmos/gov/v1/query.proto (1 hunks)
• proto/cosmos/staking/v1beta1/staking.proto (1 hunks)
• tests/e2e/auth/suite.go (2 hunks)
• x/auth/vesting/README.md (1 hunks)
• x/authz/README.md (1 hunks)
• x/gov/README.md (1 hunks)
• x/gov/types/config.go (1 hunks)
• x/group/internal/orm/sequence_property_test.go (1 hunks)
• x/nft/simulation/operations.go (2 hunks)
• x/staking/README.md (1 hunks)

Files skipped from review due to trivial changes (15)

• CHANGELOG.md
• api/cosmos/gov/v1/query.pulsar.go
• api/cosmos/staking/v1beta1/staking.pulsar.go
• docs/architecture/adr-037-gov-split-vote.md
• docs/architecture/adr-050-sign-mode-textual.md
• docs/spec/fee_distribution/f1_fee_distr.tex
• proto/cosmos/gov/v1/genesis.proto
• proto/cosmos/gov/v1/query.proto
• proto/cosmos/staking/v1beta1/staking.proto
• tests/e2e/auth/suite.go
• x/auth/vesting/README.md
• x/authz/README.md
• x/gov/types/config.go
• x/group/internal/orm/sequence_property_test.go
• x/staking/README.md

Additional comments: 6

api/cosmos/gov/v1/genesis.pulsar.go (1)

• 1243-1260: The comments have been updated to correct typographical errors and to mark certain fields as deprecated, guiding users to prefer the params field. This aligns with the PR's objective of fixing typos and updating documentation without altering the functionality of the code.

docs/build/building-modules/01-module-manager.md (1)

• 275-276: The typo in the function description of NewBasicManagerFromManager has been corrected from "Contructor" to "Constructor." This is a textual change and does not affect the functionality of the code.

x/gov/README.md (1)

• 2487-2493: The addition to the documentation provides clear guidelines on where and how metadata should be stored for proposals and votes within the governance module. It emphasizes consistency between the gov and group modules and the importance of client applications being able to rely on the structure of metadata across chains. The JSON structure provided for both proposal and vote metadata is concise and aligns with the recommendations for off-chain and on-chain storage respectively.

x/nft/simulation/operations.go (3)

• 95-98: The variable name change from recieverStr to receiverStr is consistent with the PR's objective to fix typographical errors and does not alter the logic or functionality of the code. This change improves code readability and correctness.

• 101-104: The corrected variable name receiverStr is properly used in the Receiver field of the MsgSend struct, aligning with the PR's objective to fix typographical errors.

• 92-98: > Note: This review was outside the patches, so it was mapped to the patch with the greatest overlap. Original lines [1-107]

Verify if there are any changes to proto files in this PR and ensure that the make proto-gen command has been run to generate the necessary changes, which should be included in the commit.

---

The verification process confirms that proto files have been modified in this PR, and corresponding .pb.go files have also been updated. This suggests that the make proto-gen command was likely run to generate the necessary changes, which are included in the commit.

Final review comment:

Verified that proto files have been updated and the necessary generated files are included in the commit.

[DongLieu]

It doesn't seem to work as well as codespell.

I tried the bot/fix-typo branch and there were still a lot of typos.

cc: @julienrbrt

[julienrbrt]

It doesn't seem to work as well as codespell.

I tried the bot/fix-typo branch and there were still a lot of typos.

cc: @julienrbrt

Very true, we could update the job to use codespell instead.
However, is the first run of the bot, so maybe it will find other issues at its second run tomorrow 🤷🏾‍♂️— still better to have everything resolved faster.

[DongLieu]

I understand contributors do not accept typo issues.

But what do you think of a PR that solves almost all typos?

[julienrbrt]

I understand contributors do not accept typo issues.

But what do you think of a PR that solves almost all typos?

We'll let the bot do it: #18686, but thanks for the suggestion :)