docs fix spelling issues #1027
docs fix spelling issues #1027
Conversation
WalkthroughThe changes made in this pull request focus on minor grammatical corrections and clarifications in the Changes
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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
README.md (2)
35-35: Add a comma before the conjunction.While the correction of "its" to "it's" is appropriate, consider adding a comma before "and" as it connects two independent clauses:
-The `injective-helix` is built using Nuxt and TailwindCSS and it's powered by the [injective-ts monorepo](https://github.com/InjectiveLabs/injective-ts/). +The `injective-helix` is built using Nuxt and TailwindCSS, and it's powered by the [injective-ts monorepo](https://github.com/InjectiveLabs/injective-ts/).🧰 Tools
🪛 LanguageTool
[uncategorized] ~35-~35: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...lix` is built using Nuxt and TailwindCSS and it's powered by the [injective-ts monor...(COMMA_COMPOUND_SENTENCE)
47-47: Remove the comma after the month.While the addition of "a" in "a couple of simple steps" improves grammar, the comma after "January" is unnecessary when specifying a month and year. Consider:
-We've migrated the `injective-helix` repo to Nuxt3 in January, 2023. To make the migration on your fork, there are a couple of simple steps that you have to do: +We've migrated the `injective-helix` repo to Nuxt3 in January 2023. To make the migration on your fork, there are a couple of simple steps that you have to do:🧰 Tools
🪛 LanguageTool
[uncategorized] ~47-~47: When specifying a month and year, the comma is unnecessary.
Context: ... theinjective-helixrepo to Nuxt3 in January, 2023. To make the migration on your fork, th...(COMMA_AFTER_A_MONTH)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[uncategorized] ~35-~35: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...lix` is built using Nuxt and TailwindCSS and it's powered by the [injective-ts monor...
(COMMA_COMPOUND_SENTENCE)
[style] ~37-~37: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ble and run the yarn run dev command. You can find all of the available networks ...
(ENGLISH_WORD_REPEAT_BEGINNING_RULE)
[style] ~37-~37: Consider removing “of” to be more concise
Context: ...he yarn run dev command. You can find all of the available networks (i.e - predefined se...
(ALL_OF_THE)
[uncategorized] ~47-~47: When specifying a month and year, the comma is unnecessary.
Context: ... the injective-helix repo to Nuxt3 in January, 2023. To make the migration on your fork, th...
(COMMA_AFTER_A_MONTH)
🔇 Additional comments (1)
README.md (1)
43-43: LGTM!
The sentence is clear and grammatically correct.
1 participant
Hey ! In order to ensure a better understanding I fixed several spelling issues.
Summary by CodeRabbit