-
Notifications
You must be signed in to change notification settings - Fork 225
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
fix broken links in README #353
base: main
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughThis PR updates the Changes
Possibly related PRs
Suggested reviewers
Poem
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
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 (2)
README.md (2)
16-19
: Review Comment on InitiaScan Link Update
The updated "InitiaScan" link now correctly points tohttps://scan.testnet.initia.xyz/initiation-2
, which aligns with the intended change in the service endpoint. However, note that LanguageTool flagged a potential misspelling on the descriptive text ("multi-chain explorer"). Please verify whether the intended style is "multichain" (one word) or "multi-chain" as used here.🧰 Tools
🪛 LanguageTool
[misspelling] ~16-~16: This word is normally spelled as one.
Context: ...can.testnet.initia.xyz/initiation-2): A multi-chain explorer with VM-specific tools and inf...(EN_COMPOUNDS_MULTI_CHAIN)
[style] ~19-~19: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...chain-wide on-chain identity system. - [Initia Wallet](https://chromewebstore.google.c...(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
36-37
: Review Comment on Unchanged Medium Link
The PR objective mentioned that the Medium link is broken and requires updating; however, the Medium link (https://medium.com/@initiafdn
) on line 36 remains unchanged. Please confirm whether this omission is intentional or if an update is still needed for the Medium link.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[misspelling] ~16-~16: This word is normally spelled as one.
Context: ...can.testnet.initia.xyz/initiation-2): A multi-chain explorer with VM-specific tools and inf...
(EN_COMPOUNDS_MULTI_CHAIN)
🔇 Additional comments (1)
README.md (1)
34-36
: Review Comment on Twitter Link Update
The Twitter link has been updated tohttps://x.com/initia
, reflecting the rebranding from Twitter to X. This change is correct as per the PR summary. Ensure that any related documentation (if applicable) is in sync with this update.
Some links in README are outdated, Medium link is still broken. The links on the official website also needs to be fixed, by the way.