-
Notifications
You must be signed in to change notification settings - Fork 4.1k
fix: correct various typos across codebase #25106
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
Conversation
📝 WalkthroughWalkthroughThis change set consists exclusively of minor documentation and comment corrections across various files. These include typo fixes, clarification of function and method descriptions, and updates to documentation and changelog entries. No code logic, control flow, or functionality is affected in any of the files. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this 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
🧹 Nitpick comments (1)
x/auth/vesting/README.md (1)
42-45: UsePermanentLockedto match the canonical account type nameThroughout the spec (see the TOC entry and section header), the canonical type is
PermanentLockedAccount, notPermanentVesting. Replacing the typo withPermanentLockedkeeps terminology consistent and avoids confusion.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Knowledge Base: Disabled due to Reviews > Disable Knowledge Base setting
📒 Files selected for processing (16)
client/broadcast_test.go(1 hunks)client/query.go(1 hunks)client/v2/CHANGELOG.md(1 hunks)client/v2/README.md(2 hunks)codec/codec.go(1 hunks)codec/types/any_test.go(1 hunks)docs/docs/learn/beginner/02-query-lifecycle.md(1 hunks)store/cachekv/benchmark_test.go(1 hunks)store/iavl/store.go(1 hunks)testutil/ioutil.go(1 hunks)types/handler.go(1 hunks)x/auth/ante/validator_tx_fee.go(1 hunks)x/auth/vesting/README.md(1 hunks)x/authz/authorizations.go(1 hunks)x/group/internal/orm/auto_uint64.go(2 hunks)x/group/internal/orm/index.go(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Summary
🔇 Additional comments (18)
x/group/internal/orm/index.go (2)
120-120: Typo correction looks goodThe article change from “an” to “a” improves readability; no functional impact.
148-148: Another successful typo fixSame article correction here—documentation now reads correctly.
x/group/internal/orm/auto_uint64.go (2)
85-85: Documentation polish acceptedCorrecting “an” to “a” keeps the comments consistent and clear.
106-106: Consistent article usage fixedThe final typo fix maintains documentation quality across the codebase.
testutil/ioutil.go (1)
76-76: LGTM! Typo correction improves comment accuracy.The correction from "director" to "directory" in the function comment is accurate and improves readability.
docs/docs/learn/beginner/02-query-lifecycle.md (1)
50-50: LGTM! Documentation typo correction improves clarity.The correction from "test" to "text" in the gRPC command comment is accurate and makes the documentation clearer.
store/iavl/store.go (1)
294-294: LGTM! Comment typo correction improves readability.The correction from "gatest" to "gets" fixes a clear typing error and makes the function comment grammatically correct.
codec/codec.go (1)
15-15: LGTM! Comment correction improves factual accuracy.The correction from "to Codec" to "two Codec" implementations is both grammatically correct and factually accurate, as the SDK provides exactly two Codec implementations (AminoCodec and ProtoCodec).
types/handler.go (1)
21-21: LGTM! Comment refinement removes redundancy and improves clarity.The correction removes the redundant "ChainDecorator" reference, making the comment more concise and focused on what the
ChainAnteDecoratorsfunction actually does.x/auth/ante/validator_tx_fee.go (1)
14-14: Comment typo fix looks goodThe corrected wording improves readability without affecting functionality. No further action required.
codec/types/any_test.go (1)
31-31: Typo correctedClearer comment—nothing else changed. ✅
client/broadcast_test.go (1)
23-23: Clarified test descriptionAccurate description enhances test intent. Looks good.
client/query.go (1)
63-63: Fixed comment referenceNow correctly states “fee payer”; documentation matches implementation.
x/authz/authorizations.go (1)
32-32: Minor grammar adjustment approvedSlight wording change improves clarity—with no code impact.
client/v2/CHANGELOG.md (1)
41-41: Changelog entry reads clearly after typo fixThe removal of the redundant word improves readability and accuracy.
client/v2/README.md (2)
95-99: Reference to non-existent “hubl” keyring removedThe sentence now correctly states that only the Cosmos SDK keyring implements the required interface.
258-259: Summary streamlinedThe closing summary is concise and typo-free after the edit.
store/cachekv/benchmark_test.go (1)
75-76: Comment now matches the codeRenaming “context” → “store” in the docstring eliminates confusion.
Description
client/broadcast_test.go
Test the correct code is returned when- not a complete commentclient/query.go
granter-payerclient/v2/CHANGELOG.md
parsing in when- delete extra inclient/v2/README.md
deleted references to hublcodec/codec.go
SDK provides to Codec-SDK provides two Codeccodec/types/any_test.go
tests-testdocs/docs/learn/beginner/02-query-lifecycle.md
test - text
store/cachekv/benchmark_test.go
CurrentContext-CurrentStorestore/iavl/store.go
gatest-getstestutil/ioutil.go
director-directorytypes/handler.go
ChainAnteDecorators ChainDecorator-- Removed redundantx/auth/ante/validator_tx_fee.go
can the tx priority-and the tx priorityx/auth/vesting/README.md
PermenantVesting-PermanentVestingx/authz/authorizations.go
accept and authorization-accept an authorizationx/group/internal/orm/auto_uint64.go
x/group/internal/orm/index.go
an MultiKeyIndex-a MultiKeyIndexSummary by CodeRabbit
Documentation
Style