-
Notifications
You must be signed in to change notification settings - Fork 32
chore: fix some minor issues in comments #117
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: stellrust <[email protected]>
WalkthroughThis update makes minor textual corrections across three markdown documents. The changes involve fixing typographical errors, removing redundant words, and eliminating unnecessary spaces to enhance readability. No functional, technical, or logical modifications were made to any code or documentation structure. Changes
Estimated code review effort1 (~2 minutes) Poem
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. 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 (4)
relaying/Readme.md (1)
90-90: Replace the hard-tab with spaces.Line 90 starts with a tab character, tripping
markdownlint(MD010).
Convert it to two spaces to stay consistent with surrounding list items.- * Samsung 980 Pro <- recommended for reliability and availability. we use these on-site. + * Samsung 980 Pro <- recommended for reliability and availability. we use these on-site.swe/starport.md (1)
342-342: Minor wording tweak (optional).You also corrected “verisons → versions”. Consider tightening the sentence by dropping the second “very” to avoid redundancy, but feel free to keep as-is.
No code change required; note is purely stylistic.
incidents/riseup-for-adi.md (2)
82-82: Remove leading hard-tab.Line 82 begins with a tab, violating MD010. Replace it with spaces to align with the rest of the document.
- The osmosis validator set is treated as a part of the team. There's no way to look at the situation with the ICF and say that is the case for the Cosmos hub. + The osmosis validator set is treated as a part of the team. There's no way to look at the situation with the ICF and say that is the case for the Cosmos hub.
144-145: Strip hard-tab at paragraph start.Same MD010 issue here—convert the tab to spaces.
- Look I'm a person who has failed a lot. I have failed at various and sundry things and succeeded in a couple of things which is fairly normal for entrepreneurs. Bucky ICF is not going well for you. I see in your commentary online, the exact things that are causing the ICF to have problems. Like why is the ICF having difficulty? Well, you have created a bit of a buck passing culture. The people who are fighting against that buck passing culture inside of IG, I don't feel that they're getting the necessary support and I feel that is ultimately what caused the recent difficulties surrounding the choice of SDK used by CW. + Look I'm a person who has failed a lot. I have failed at various and sundry things and succeeded in a couple of things which is fairly normal for entrepreneurs. Bucky ICF is not going well for you. I see in your commentary online, the exact things that are causing the ICF to have problems. Like why is the ICF having difficulty? Well, you have created a bit of a buck passing culture. The people who are fighting against that buck passing culture inside of IG, I don't feel that they're getting the necessary support and I feel that is ultimately what caused the recent difficulties surrounding the choice of SDK used by CW.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
incidents/riseup-for-adi.md(2 hunks)relaying/Readme.md(1 hunks)swe/starport.md(2 hunks)
🧰 Additional context used
🪛 LanguageTool
swe/starport.md
[style] ~3-~3: Consider using a different adverb to strengthen your wording.
Context: ...ngineering practice on starport and are very grateful. Lately, dx has taken a turn...
(VERY_DEEPLY)
🪛 markdownlint-cli2 (0.17.2)
incidents/riseup-for-adi.md
82-82: Hard tabs
Column: 1
(MD010, no-hard-tabs)
144-144: Hard tabs
Column: 1
(MD010, no-hard-tabs)
relaying/Readme.md
90-90: Hard tabs
Column: 1
(MD010, no-hard-tabs)
🔇 Additional comments (1)
swe/starport.md (1)
3-3: Typo fix looks good.The “beleive → believe” correction improves readability with no side-effects.
fix some minor issues in comments
Summary by CodeRabbit