-
Notifications
You must be signed in to change notification settings - Fork 59
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
Minor updates to learn section #379
Conversation
Warning Rate Limit Exceeded@jcstein has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 1 minutes and 36 seconds before requesting another review. How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. WalkthroughThe recent updates involve restructuring the sidebar items under the "Learn" section in 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.
Actionable comments posted: 1
Out of diff range and nitpick comments (2)
learn/specifications.md (1)
Line range hint
5-6
: Remove multiple consecutive blank lines.- - +learn/about.md (1)
Line range hint
2-2
: Fix grammatical issue.- Our mission is allow anyone to launch their own sovereign, customizable blockchain as easily as a smart contract by utilizing the shared security of a modular data availability network. + Our mission is to allow anyone to launch their own sovereign, customizable blockchain as easily as a smart contract by utilizing the shared security of a modular data availability network.
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (5)
- .vitepress/config.ts (1 hunks)
- learn/about.md (2 hunks)
- learn/intro.md (2 hunks)
- learn/specifications.md (1 hunks)
- learn/stack.md (1 hunks)
Files skipped from review due to trivial changes (1)
- learn/stack.md
Additional comments not posted (5)
learn/about.md (2)
Line range hint
15-88
: LGTM!
84-88
: LGTM!.vitepress/config.ts (3)
Line range hint
1-187
: LGTM!
188-210
: LGTM!
Line range hint
211-278
: LGTM!
.vitepress/config.ts
Outdated
text: "Resources", | ||
collapsed: true, | ||
items: [ | ||
{ text: "Technical Specifications", link: "/learn/specifications" }, |
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.
why not just an external link?
{ text: "Transaction flow", link: "/learn/transaction-flow" }, | ||
], | ||
}, | ||
{ |
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.
putting this in its own category with nothing else seems like it will bury it
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 UI
Review profile: CHILL
Files selected for processing (1)
- .vitepress/config.ts (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- .vitepress/config.ts
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
Out of diff range and nitpick comments (6)
learn/about.md (6)
Line range hint
3-3
: Consider using either the past participle “allowed” or the present participle “allowing” here.- Our mission is allow anyone to launch their own sovereign, customizable blockchain... + Our mission is to allow anyone to launch their own sovereign, customizable blockchain...
Line range hint
5-5
: Consider a shorter alternative to avoid wordiness.- In order to use Rollkit, it's important to have some familiarity with Cosmos SDK... + To use Rollkit, familiarity with Cosmos SDK is important...
Line range hint
33-33
: Qualifiers like “somewhat” can weaken your message and make your writing sound uncertain. Consider removing it or choosing an alternative to sound more confident.- While it's somewhat true, deploying a new layer 1 chain presents a complex set of challenges... + While it's true, deploying a new layer 1 chain presents a complex set of challenges...
Line range hint
36-36
: Consider a shorter alternative to avoid wordiness.- In order to secure the network, developers must bootstrap a sufficiently secure set of validators... + To secure the network, developers must bootstrap a sufficiently secure set of validators...
Line range hint
63-63
: ‘Many different ways’ might be wordy. Consider a shorter alternative.- Rollkit could be used in many different ways... + Rollkit could be used in various ways...
Line range hint
67-67
: Consider replacing ‘gives’ with a different word to let your writing stand out.- Rollkit gives developers the flexibility to use pre-existing ABCI-compatible state machines... + Rollkit provides developers the flexibility to use pre-existing ABCI-compatible state machines...
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- learn/about.md (2 hunks)
Additional Context Used
LanguageTool (6)
learn/about.md (6)
Near line 3: Consider using either the past participle “allowed” or the present participle “allowing” here.
Context: # About Rollkit Our mission is allow anyone to launch their own sovereign, c...
Near line 5: Consider a shorter alternative to avoid wordiness.
Context: ...lar data availability network. ::: tip In order to use Rollkit, it's important to have som...
Near line 33: Qualifiers like “somewhat” can weaken your message and make your writing sound uncertain. Consider removing it or choosing an alternative to sound more confident.
Context: ...r the problems listed above. While it's somewhat true, deploying a new layer 1 chain pre...
Near line 36: Consider a shorter alternative to avoid wordiness.
Context: ... barrier to entry for some developers. In order to secure the network, developers must boo...
Near line 63: ‘many different ways’ might be wordy. Consider a shorter alternative.
Context: ...ntioned above, Rollkit could be used in many different ways. From sovereign rollups, to settlement ...
Near line 67: Consider replacing ‘gives’ with a different word to let your writing stand out.
Context: ...with any execution environment Rollkit gives developers the flexibility to use pre-existing ABC...
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Summary by CodeRabbit
New Features
Improvements
Content Updates