-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix: fix broken link #23737
base: main
Are you sure you want to change the base?
fix: fix broken link #23737
Conversation
📝 WalkthroughWalkthroughThis pull request updates multiple documentation files by removing or updating external reference links. Changes include the deletion of URL references to blog posts, external articles, and additional guides, as well as corrections to the Cosmos Whitepaper URL. No modifications affect functionality, structure, or public entity declarations. Changes
Suggested labels
Suggested reviewers
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 using PR comments)
Other keywords and placeholders
Documentation and Community
|
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)
tools/cosmovisor/CHANGELOG.md (1)
168-169
: Broken Link Removal and Sentence Enhancement:
The removal of the broken link referencing the README file (previously included in the v1.0.0 entry) has been handled appropriately, which resolves the link issue described in the PR. Additionally, the new sentence"This is the first release and we started this changelog on 2021-07-01."
contains two independent clauses that would benefit from a comma before "and" for improved readability. Consider revising it to:
"This is the first release, and we started this changelog on 2021-07-01."
🧰 Tools
🪛 LanguageTool
[uncategorized] ~168-~168: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... - 2021-08-06 This is the first release and we started this changelog on 2021-07-01...(COMMA_COMPOUND_SENTENCE)
📜 Review details
Configuration used: .coderabbit.yml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
docs/architecture/README.md
(0 hunks)docs/architecture/adr-019-protobuf-state-encoding.md
(0 hunks)docs/architecture/adr-029-fee-grant-module.md
(0 hunks)docs/build/building-apps/04-security-part-1.md
(1 hunks)store/snapshots/README.md
(0 hunks)store/v2/snapshots/README.md
(0 hunks)tools/cosmovisor/CHANGELOG.md
(1 hunks)x/gov/README.md
(1 hunks)x/slashing/README.md
(1 hunks)x/staking/README.md
(1 hunks)x/upgrade/README.md
(0 hunks)
💤 Files with no reviewable changes (6)
- docs/architecture/adr-029-fee-grant-module.md
- x/upgrade/README.md
- store/snapshots/README.md
- docs/architecture/adr-019-protobuf-state-encoding.md
- docs/architecture/README.md
- store/v2/snapshots/README.md
✅ Files skipped from review due to trivial changes (4)
- x/slashing/README.md
- x/gov/README.md
- x/staking/README.md
- docs/build/building-apps/04-security-part-1.md
🧰 Additional context used
📓 Path-based instructions (1)
`**/*.md`: "Assess the documentation for misspellings, gramm...
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
tools/cosmovisor/CHANGELOG.md
🪛 LanguageTool
tools/cosmovisor/CHANGELOG.md
[uncategorized] ~168-~168: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ... - 2021-08-06 This is the first release and we started this changelog on 2021-07-01...
(COMMA_COMPOUND_SENTENCE)
⏰ Context from checks skipped due to timeout of 90000ms (2)
- GitHub Check: main
- GitHub Check: Summary
Description
Closes: #XXXX
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
in the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
Please see Pull Request Reviewer section in the contributing guide for more information on how to review a pull request.
I have...
Summary by CodeRabbit