-
Notifications
You must be signed in to change notification settings - Fork 646
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
docs: fix typo #6455
docs: fix typo #6455
Conversation
WalkthroughWalkthroughThe changes across various versions of the Changes
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? TipsChatThere are 3 ways to chat with CodeRabbit:
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)
Additionally, you can add CodeRabbit Configration 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.
Thanks, @healthyyyoung!
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
Review details
Configuration used: .coderabbit.yml
Review profile: CHILL
Files selected for processing (4)
- docs/versioned_docs/version-v4.6.x/01-ibc/02-integration.md (1 hunks)
- docs/versioned_docs/version-v5.4.x/01-ibc/02-integration.md (1 hunks)
- docs/versioned_docs/version-v6.3.x/01-ibc/02-integration.md (1 hunks)
- docs/versioned_docs/version-v7.5.x/01-ibc/02-integration.md (1 hunks)
Additional context used
Path-based instructions (4)
docs/versioned_docs/version-v4.6.x/01-ibc/02-integration.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/versioned_docs/version-v5.4.x/01-ibc/02-integration.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/versioned_docs/version-v6.3.x/01-ibc/02-integration.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/versioned_docs/version-v7.5.x/01-ibc/02-integration.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
LanguageTool
docs/versioned_docs/version-v4.6.x/01-ibc/02-integration.md
[uncategorized] ~25-~25: When ‘Set-up’ is used as a noun or modifier, it needs to be hyphenated.
Context: ...eKeysand initialize their
Keepers` - Set up corresponding routers and routes for th...
[style] ~81-~81: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...ibc, and
x/ibc-transfer` modules), we need to grant specific capabilities through the...
[style] ~117-~117: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nues } ``` ### RegisterRouters
IBC needs to know which module is bound to which por...
[uncategorized] ~127-~127: Possible missing comma found.
Context: ... or a packet. Currently, aRouter
is static so it must be initialized and set corre...
[style] ~146-~146: Consider a shorter alternative to avoid wordiness.
Context: ... .. continues ``` ### Module Managers In order to use IBC, we need to add the new modules...
[uncategorized] ~178-~178: Possible missing comma found.
Context: ...theHeader
andValidatorSet
of this chain which is stored at each height during t...
[style] ~180-~180: Consider a shorter alternative to avoid wordiness.
Context: ...ast historical info at any given height in order to verify the light client `ConsensusState...
[uncategorized] ~184-~184: Possible missing comma found.
Context: ...-client/abci.go) logic as well. This is optional as it is only required if your applicat...
[style] ~220-~220: As a shorter alternative for ‘able to’, consider using “can now”.
Context: ...ou have now wired up the IBC module and are now able to send fungible tokens across different c...
[uncategorized] ~221-~221: Possible missing comma found.
Context: ... you want to have a broader view of the changes take a look into the SDK's [SimApp
](h...docs/versioned_docs/version-v5.4.x/01-ibc/02-integration.md
[uncategorized] ~25-~25: When ‘Set-up’ is used as a noun or modifier, it needs to be hyphenated.
Context: ...eKeysand initialize their
Keepers` - Set up corresponding routers and routes for th...
[style] ~81-~81: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...ibc, and
x/ibc-transfer` modules), we need to grant specific capabilities through the...
[style] ~117-~117: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nues } ``` ### RegisterRouters
IBC needs to know which module is bound to which por...
[style] ~146-~146: Consider a shorter alternative to avoid wordiness.
Context: ... .. continues ``` ### Module Managers In order to use IBC, we need to add the new modules...
[uncategorized] ~178-~178: Possible missing comma found.
Context: ...theHeader
andValidatorSet
of this chain which is stored at each height during t...
[style] ~180-~180: Consider a shorter alternative to avoid wordiness.
Context: ...ast historical info at any given height in order to verify the light client `ConsensusState...
[style] ~220-~220: As a shorter alternative for ‘able to’, consider using “can now”.
Context: ...ou have now wired up the IBC module and are now able to send fungible tokens across different c...
[uncategorized] ~221-~221: A comma might be missing here.
Context: ... you want to have a broader view of the changes take a look into the SDK's [SimApp
](h...docs/versioned_docs/version-v6.3.x/01-ibc/02-integration.md
[uncategorized] ~25-~25: When ‘Set-up’ is used as a noun or modifier, it needs to be hyphenated.
Context: ...eKeysand initialize their
Keepers` - Set up corresponding routers and routes for th...
[style] ~81-~81: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...ibc, and
x/ibc-transfer` modules), we need to grant specific capabilities through the...
[style] ~117-~117: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nues } ``` ### RegisterRouters
IBC needs to know which module is bound to which por...
[uncategorized] ~127-~127: Possible missing comma found.
Context: ... or a packet. Currently, aRouter
is static so it must be initialized and set corre...
[style] ~146-~146: Consider a shorter alternative to avoid wordiness.
Context: ... .. continues ``` ### Module Managers In order to use IBC, we need to add the new modules...
[style] ~180-~180: Consider a shorter alternative to avoid wordiness.
Context: ...ast historical info at any given height in order to verify the light client `ConsensusState...
[uncategorized] ~184-~184: Possible missing comma found.
Context: ...-client/abci.go) logic as well. This is optional as it is only required if your applicat...
[style] ~220-~220: As a shorter alternative for ‘able to’, consider using “can now”.
Context: ...ou have now wired up the IBC module and are now able to send fungible tokens across different c...
[uncategorized] ~221-~221: Possible missing comma found.
Context: ... you want to have a broader view of the changes take a look into the SDK's [SimApp
](h...docs/versioned_docs/version-v7.5.x/01-ibc/02-integration.md
[style] ~40-~40: ‘is in contrast to’ might be wordy. Consider a shorter alternative.
Context: ... follow the steps listed below. This is in contrast to earlier versions of ibc-go when `07-ten...
[style] ~138-~138: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nues } ``` ### RegisterRouters
IBC needs to know which module is bound to which por...
[uncategorized] ~148-~148: Possible missing comma found.
Context: ... or a packet. Currently, aRouter
is static so it must be initialized and set corre...
[style] ~167-~167: Consider a shorter alternative to avoid wordiness.
Context: ... .. continues ``` ### Module Managers In order to use IBC, we need to add the new modules...
[uncategorized] ~199-~199: Possible missing comma found.
Context: ...theHeader
andValidatorSet
of this chain which is stored at each height during t...
[style] ~201-~201: Consider a shorter alternative to avoid wordiness.
Context: ...ast historical info at any given height in order to verify the light client `ConsensusState...
[style] ~233-~233: As a shorter alternative for ‘able to’, consider using “can now”.
Context: ...ou have now wired up the IBC module and are now able to send fungible tokens across different c...
[uncategorized] ~234-~234: A comma might be missing here.
Context: ... you want to have a broader view of the changes take a look into the SDK's [SimApp
](h...
Markdownlint
docs/versioned_docs/version-v4.6.x/01-ibc/02-integration.md
8-8: null
Multiple top-level headings in the same documentdocs/versioned_docs/version-v5.4.x/01-ibc/02-integration.md
8-8: null
Multiple top-level headings in the same documentdocs/versioned_docs/version-v6.3.x/01-ibc/02-integration.md
8-8: null
Multiple top-level headings in the same documentdocs/versioned_docs/version-v7.5.x/01-ibc/02-integration.md
12-12: Expected: 0 or 2; Actual: 1
Trailing spaces
39-39: Expected: 0 or 2; Actual: 1
Trailing spaces
9-9: null
Multiple top-level headings in the same document
Additional comments not posted (4)
docs/versioned_docs/version-v4.6.x/01-ibc/02-integration.md (1)
135-135
: Corrected typo in the comment.docs/versioned_docs/version-v5.4.x/01-ibc/02-integration.md (1)
135-135
: Corrected typo in the comment.docs/versioned_docs/version-v6.3.x/01-ibc/02-integration.md (1)
135-135
: Corrected typo in the comment.docs/versioned_docs/version-v7.5.x/01-ibc/02-integration.md (1)
156-156
: Corrected typo in the comment.
Description
closes: #XXXX
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.
docs/
).godoc
comments.Files changed
in the GitHub PR explorer.SonarCloud Report
in the comment section below once CI passes.Summary by CodeRabbit