Skip to content
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 obvious issues with tutorials #347

Merged
merged 3 commits into from
Apr 23, 2024
Merged

Fix obvious issues with tutorials #347

merged 3 commits into from
Apr 23, 2024

Conversation

yarikbratashchuk
Copy link
Contributor

@yarikbratashchuk yarikbratashchuk commented Apr 18, 2024

Passed beginner and intermediate tutorials and spotted some discrepancies and outdated code.

Summary by CodeRabbit

  • New Features

    • Updated the initialization script for Arabica testnet to use a new RPC port.
    • Enhanced various tutorials with additional tips and updated instructions for tools and commands, improving user setup and operational guidance.
    • Transitioned from npm to yarn for installing dependencies in the frontend tutorial.
  • Documentation

    • Added a tip for installing gcc on Linux VMs in the CosmWasm tutorial.
    • Updated the IBC connection tutorial with revised instructions for setting the channel hash.
    • Improved the Recipe Book tutorial with updated import paths and initialization practices.
  • Bug Fixes

    • Enhanced error handling and updated error messages in the Wordle tutorial to improve clarity and user experience.

Copy link
Contributor

coderabbitai bot commented Apr 18, 2024

Walkthrough

The recent updates enhance user experience and functionality across several tutorials and scripts, focusing on improving installation processes, updating command lines and ports, and refining documentation. Key changes include adjusting RPC ports, switching from npm to yarn, refining error handling, and updating usage instructions and paths for better clarity and performance.

Changes

Files Change Summary
scripts/gm/init-arabica-testnet.sh Changed RPC port from 36657 to 26657.
tutorials/cosmwasm.md Added gcc installation tip; updated transaction query command.
tutorials/gm-world-...-testnet.md Instructed to keep node running during tutorial.
tutorials/gm-world-frontend.md Switched from npm to yarn for global installations; added installation tip for yarn.
tutorials/ibc-connection.md Updated instructions for setting channel hash in IBC tutorial.
tutorials/recipe-book.md Updated import paths, introduced storeAdapter, and added key creation and funding tips.
tutorials/wordle.md Updated URLs, added error handling, and modified error messages.

🐇💻
In the garden of code, where the bits all roam,
A rabbit hopped by, making changes its home.
Ports and paths, with a yarn not a npm,
Crafting the web, with a hop, skip, and vim.
Oh, how the digital meadows do bloom,
With each line of code and each crafty tune! 🌼🎶


Recent Review Details

Configuration used: CodeRabbit UI

Commits Files that changed from the base of the PR and between 83875a8 and 069c7ff.
Files selected for processing (1)
  • tutorials/wordle.md (8 hunks)
Additional Context Used
LanguageTool (126)
tutorials/wordle.md (126)

Near line 5: Possible spelling mistake found.
Context: ... tutorial guide will go over building a cosmos-sdk app for Rollkit, the Sovereign-Rollup i...


Near line 6: Possible spelling mistake found.
Context: ...l go over building a cosmos-sdk app for Rollkit, the Sovereign-Rollup implementation of...


Near line 6: Possible spelling mistake found.
Context: ...lding a cosmos-sdk app for Rollkit, the Sovereign-Rollup implementation of CometBFT, for the pop...


Near line 6: Possible spelling mistake found.
Context: ... the Sovereign-Rollup implementation of CometBFT, for the popular game [Wordle](https://...


Near line 9: The verb ‘set up’ is spelled as two words. The noun ‘setup’ is spelled as one.
Context: ...ml). This tutorial will go over how to setup Rollkit in the Ignite CLI and use it to...


Near line 9: Possible spelling mistake found.
Context: ...This tutorial will go over how to setup Rollkit in the Ignite CLI and use it to build t...


Near line 16: Possible spelling mistake found.
Context: ...s tutorial will explore developing with Rollkit, which is still in Alpha stage. If you ...


Near line 17: The official name of this software platform is spelled with a capital “H”.
Context: ...e. If you run into bugs, please write a Github [Issue ticket](https://github.com/rollk...


Near line 21: Possible spelling mistake found.
Context: ...e/rollkit). Learn how to restart your rollup. ::: ::: warning T...


Near line 25: Possible spelling mistake found.
Context: ...e script for this tutorial is built for Celestia's [Arabica devnet](https://docs.celesti...


Near line 26: Possible spelling mistake found.
Context: ...torial is built for Celestia's [Arabica devnet](https://docs.celestia.org/nodes/arabic...


Near line 37: Unpaired symbol: ‘’ seems to be missing
Context: ...rld
* Recipe Book or [Blog and Mo...


Near line 37: Unpaired symbol: ‘]’ seems to be missing
Context: ...Recipe Book](/tutorials/recipe-book) or [Blog and Module Basics](https://docs.ign...


Near line 38: Possible spelling mistake found.
Context: ...://docs.ignite.com/guide/blog/intro) * [Nameservice Tutorial](https://docs.ignite.com/v0.25...


Near line 41: Consider a shorter alternative to avoid wordiness.
Context: ...ge) You do not have to do those guides in order to follow this Wordle tutorial, but doing ...


Near line 58: Possible missing comma found.
Context: ...ght means it’s a correct letter for the Wordle included in the wrong position. A grey ...


Near line 59: Possible spelling mistake. ‘grey’ is British English.
Context: ...ordle included in the wrong position. A grey highlight means the letter isn’t part o...


Near line 78: Possible spelling mistake found.
Context: ...ment. ## ⛓️ Ignite and scaffolding the wordle chain {#ignite-scaffold-wordle-chain} <...


Near line 83: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...--> ### 🔥 Ignite {#ignite} Ignite is an amazing CLI tool to help us get started buildin...


Near line 84: Possible spelling mistake found.
Context: ...tarted building our own blockchains for cosmos-sdk apps. It provides lots of power tooling...


Near line 85: Possible spelling mistake found.
Context: ...mos-sdk apps. It provides lots of power toolings and scaffoldings for adding messages, t...


Near line 85: Possible spelling mistake found.
Context: ... It provides lots of power toolings and scaffoldings for adding messages, types, and modules...


Near line 86: Possible spelling mistake found.
Context: ...ages, types, and modules with a host of cosmos-sdk libraries provided. You can read more ...


Near line 97: The preposition “on” seems more likely in this position than the preposition “in”.
Context: ...ocal/bin/ ``` This installs Ignite CLI in your local machine. This tutorial uses ...


Near line 98: The operating system from Apple is written “macOS”.
Context: ...our local machine. This tutorial uses a MacOS but it should work for Windows. For Win...


Near line 98: Use a comma before ‘but’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...ocal machine. This tutorial uses a MacOS but it should work for Windows. For Windows...


Near line 111: Possible missing comma found.
Context: ...p ``` You should see an output of help commands meaning Ignite was installed successful...


Near line 114: Possible spelling mistake found.
Context: ...d successfully! ### ⛓️ Scaffolding the wordle chain {#scaffolding-wordle-chain} Now,...


Near line 129: The verb ‘set up’ is spelled as two words. The noun ‘setup’ is spelled as one.
Context: ... We first run the following command to setup our project for our new blockchain, Wor...


Near line 136: Possible spelling mistake found.
Context: ... scaffolds a new chain directory called wordle in your local directory from which you...


Near line 150: Possible spelling mistake found.
Context: ...l directories and architecture for your cosmos-sdk blockchain. | File/directory | Purpose...


Near line 152: Possible typo: you repeated a whitespace
Context: ... blockchain. | File/directory | Purpose | | -------------- | -------------------...


Near line 154: Possible typo: you repeated a whitespace
Context: ...------------------------------- | | app/ | Files that wire together the blockchai...


Near line 154: You might be missing the article “the” here.
Context: ...mportant file is app.go that contains type definition of the blockchain and functi...


Near line 155: Possible typo: you repeated a whitespace
Context: ...ns to create and initialize it. | | cmd/ | The main package responsible for the C...


Near line 155: Possible typo: you repeated a whitespace
Context: ...onsible for the CLI of compiled binary. | | docs/ | Directory for proje...


Near line 156: Possible typo: you repeated a whitespace
Context: ... | | docs/ | Directory for project documentation. B...


Near line 156: Possible spelling mistake found.
Context: ...r project documentation. By default, an OpenAPI spec is generated. ...


Near line 156: Possible typo: you repeated a whitespace
Context: ... default, an OpenAPI spec is generated. | | proto/ | Protocol buffer fil...


Near line 157: Possible typo: you repeated a whitespace
Context: ... | | proto/ | Protocol buffer files describing the d...


Near line 157: Possible typo: you repeated a whitespace
Context: ...er files describing the data structure. | | testutil/ | Helper functions fo...


Near line 158: Possible spelling mistake found.
Context: ... | | testutil/ | Helper functions for testing. ...


Near line 158: Possible typo: you repeated a whitespace
Context: ... | | testutil/ | Helper functions for testing. ...


Near line 158: Possible typo: you repeated a whitespace
Context: ...l/ | Helper functions for testing. | | vue/ | A Vue 3 web app tem...


Near line 159: Possible spelling mistake found.
Context: ... | | vue/ | A Vue 3 web app template. ...


Near line 159: Possible typo: you repeated a whitespace
Context: ... | | vue/ | A Vue 3 web app template. ...


Near line 159: Possible typo: you repeated a whitespace
Context: .../ | A Vue 3 web app template. | | x/ | Cosmos SDK modules ...


Near line 160: Possible typo: you repeated a whitespace
Context: ... | | x/ | Cosmos SDK modules and custom modules....


Near line 160: Possible typo: you repeated a whitespace
Context: ... Cosmos SDK modules and custom modules. | | config.yml | A configuration fil...


Near line 161: Possible typo: you repeated a whitespace
Context: ... | | config.yml | A configuration file for customizing a...


Near line 161: Possible typo: you repeated a whitespace
Context: ...for customizing a chain in development. | | readme.md | A readme file for y...


Near line 162: Possible typo: you repeated a whitespace
Context: ... | | readme.md | A readme file for your sovereign appli...


Near line 162: Possible spelling mistake found.
Context: ... | | readme.md | A readme file for your sovereign application-spe...


Near line 169: Possible spelling mistake found.
Context: ...e the x directory. ## 🛠️ Setting up Rollkit {#setting-up-rollkit} Before we contin...


Near line 171: Possible spelling mistake found.
Context: ...lding our Wordle App, we need to set up Rollkit on our codebase. ### 🗞️ Installing Ro...


Near line 174: Possible spelling mistake found.
Context: ...it on our codebase. ### 🗞️ Installing Rollkit {#installing-rollkit} Run the followin...


Near line 176: Possible spelling mistake found.
Context: ...} Run the following command inside the wordle directory. ```bash go mod edit -repla...


Near line 184: Possible spelling mistake found.
Context: ...go mod download ``` With that, we have Rollkit changes added to the project directory....


Near line 187: Possible spelling mistake found.
Context: ...uild the Wordle app! ## ✨ Creating the wordle module {#creating-wordle-module} For t...


Near line 212: Possible spelling mistake found.
Context: ...kinds of accounts (notably delegating/undelegating for vesting accounts). It exposes sev...


Near line 214: Possible missing comma found.
Context: ...ies for secure interaction with other modules which must alter user balances. We bui...


Near line 238: Possible spelling mistake found.
Context: ...sages with ignite. * The first one is: SubmitWordle and it only passes the Wordle of the D...


Near line 239: Possible spelling mistake found.
Context: ...Wordle of the Day. * The second one is: SubmitGuess and it attempts to guess the submitted...


Near line 240: Possible spelling mistake found.
Context: ...nd it attempts to guess the submitted wordle. It also passes a word as a guess. Wit...


Near line 246: Possible spelling mistake found.
Context: ...{#scaffolding-a-message} To create the SubmitWordle message, we run the following command:...


Near line 252: Possible spelling mistake found.
Context: ...ubmit-wordle word ``` This creates the submit-wordle message that takes in `word` as a para...


Near line 254: Possible spelling mistake found.
Context: ...eter. We now create the final message, SubmitGuess: ```bash ignite scaffold message subm...


Near line 267: Possible spelling mistake found.
Context: ...ssages we created. ### 🏗️ Scaffolding wordle types {#scaffolding-wordle-types} ```b...


Near line 275: Consider using “who” when you are referring to a person instead of an object.
Context: ...submitter` is the address of the person that submitted the Wordle. The second type ...


Near line 294: Possible typo: you repeated a whitespace
Context: ...DK module is a piece called the keeper. The keeper handles interactions with the...


Near line 305: Possible spelling mistake found.
Context: ...gic for each message we create. ### 📤 SubmitWordle function {#submitwordle-function} We ...


Near line 307: Possible spelling mistake found.
Context: ...rdle-function} We first start with the SubmitWordle function. Open up the following file:...


Near line 385: Possible spelling mistake found.
Context: ... } } return true } ``` Here in the SubmitWordle Keeper function, we are doing a few th...


Near line 388: The grammatical number of this noun doesn’t look right. Consider replacing it.
Context: ...e Day is 5 letters long and only uses alphabets. That means no integers can be submit...


Near line 390: Possible missing comma found.
Context: ... We then create a hash from the current day the moment the Wordle was submitted. ...


Near line 390: Possible missing preposition found.
Context: ...then create a hash from the current day the moment the Wordle was submitted. We s...


Near line 396: Possible spelling mistake found.
Context: ...dy been submitted. Remember, only one wordle can be submitted per day. Everyone el...


Near line 397: Possible spelling mistake found.
Context: ...veryone else has to guess the submitted wordle. * We also have a helper function in th...


Near line 401: Possible spelling mistake found.
Context: ... contains alphabet characters. ### ⌛ SubmitGuess function {#submitguess-function} The ...


Near line 510: You might be missing the article “the” here.
Context: ... day. * Next we create a hash string of current day and the Submitter. This allows us...


Near line 510: Possible typo: you repeated a whitespace
Context: ...tring of current day and the Submitter. This allows us to create a Guess type wi...


Near line 513: Possible spelling mistake found.
Context: ...y and an address wants to guess the new wordle of the day. The index setup ensures t...


Near line 513: Possible typo: you repeated a whitespace
Context: ...nts to guess the new wordle of the day. The index setup ensures they can continu...


Near line 514: Possible spelling mistake found.
Context: ...nsures they can continue guessing a new wordle every day up to the max of 6 tries pe...


Near line 515: Possible missing comma found.
Context: ... continue guessing a new wordle every day up to the max of 6 tries per day. * We ...


Near line 517: Possible spelling mistake found.
Context: ...ss type for the Submitter for today’s wordle did reach 6 counts. If it hasn’t, we in...


Near line 517: Possible typo: you repeated a whitespace
Context: .... If it hasn’t, we increment the count. We then check if the guess is correct. W...


Near line 521: Possible spelling mistake found.
Context: ...the updated count to the state. ### 📦 Protobuf file {#protobuf-file} A few files ne...


Near line 525: Possible spelling mistake found.
Context: ...odified for this to work. The first is proto/wordle/wordle/tx.proto. Inside this file, fi...


Near line 527: Possible spelling mistake found.
Context: .... Inside this file, fill in the empty MsgSubmitGuessResponse` with the following code: ```go title=...


Near line 539: Possible spelling mistake found.
Context: ...d_keepers.go` Here, we need to add the SendCoins method to the BankKeeper interface in o...


Near line 539: Possible spelling mistake found.
Context: ...need to add the SendCoins method to the BankKeeper interface in order to allow sending the...


Near line 540: Consider a shorter alternative to avoid wordiness.
Context: ...oins method to the BankKeeper interface in order to allow sending the reward to the right g...


Near line 551: Possible spelling mistake found.
Context: ...it out for a test drive. ## ⛓️ Run the wordle chain <!-- markdownlint-disable MD013 -...


Near line 554: Possible spelling mistake found.
Context: ...ownlint-disable MD013 --> ### 🪶 Run a Celestia light node {#run-celestia-light-node} ...


Near line 556: Possible spelling mistake found.
Context: ... instructions to install and start your Celestia Data Availability layer Light Node sele...


Near line 557: Possible spelling mistake found.
Context: ... layer Light Node selecting the Arabica Devnet. You can find instructions to install a...


Near line 560: Possible spelling mistake found.
Context: ...e Go and Ignite CLI installed, and your Celestia Light Node running on your machine, you...


Near line 562: Possible spelling mistake found.
Context: ...ld, test, and launch your own sovereign rollup. Be sure you have initialized your nod...


Near line 572: Possible spelling mistake found.
Context: ...e Go and Ignite CLI installed, and your Celestia Light Node running on your machine, you...


Near line 574: Possible spelling mistake found.
Context: ...ld, test, and launch your own sovereign rollup. ### 🟢 Building and running wordle ch...


Near line 576: Possible spelling mistake found.
Context: ...gn rollup. ### 🟢 Building and running wordle chain {#build-and-run-wordle-chain} We...


Near line 591: Possible spelling mistake found.
Context: ...nit.sh` script to initialize our Wordle Rollup. You can view the contents of the scri...


Near line 594: Possible spelling mistake found.
Context: ...ipt to see how we initialize the Wordle Rollup. You can initialize the script with th...


Near line 602: Possible spelling mistake found.
Context: ...sh bash init.sh ``` With that, we have kickstarted our wordled network! Find [your acco...


Near line 602: Possible spelling mistake found.
Context: ...``` With that, we have kickstarted our wordled network! Find [your account address o...


Near line 605: Possible spelling mistake found.
Context: ...ress on an Arabica explorer to see your PayForBlobs transactions](https://explorer.modular...


Near line 616: Possible spelling mistake found.
Context: ...rrors. With Rollkit as a replacement to CometBFT, we need to wait for Celestia's Data-...


Near line 617: Possible spelling mistake found.
Context: ...ment to CometBFT, we need to wait for Celestia's Data-Availability network to ensure a...


Near line 619: Possible spelling mistake found.
Context: ...ding to the next block. Currently, in Rollkit, the single aggregator is not moving fo...


Near line 620: Possible typo: you repeated a whitespace
Context: ...it the current block to the DA network. In the future, with leader selection, bl...


Near line 621: This verb may not be in the correct tense. Consider changing the tense to fit the context better.
Context: ...ection, block production and sync logic improves dramatically. This will ask you to c...


Near line 710: Possible spelling mistake found.
Context: ...r confirming the transaction, query the txhash given the same way you did above. You ...


Near line 722: Possible spelling mistake found.
Context: ...r confirming the transaction, query the txhash given the same way you did above. You ...


Near line 726: Possible spelling mistake found.
Context: ... characters. Now try to submit another wordle even though one was already submitted ...


Near line 734: Possible spelling mistake found.
Context: ... transactions and confirming, query the txhash given the same way you did above. You ...


Near line 735: Possible spelling mistake found.
Context: ...did above. You will get an error that a wordle has already been submitted for the day....


Near line 738: When ‘five-letter’ is used as a modifier, it is usually spelled with a hyphen.
Context: ... for the day. Now let’s try to guess a five letter word: <!-- markdownlint-disable MD013 ...


Near line 746: Possible spelling mistake found.
Context: ... transactions and confirming, query the txhash given the same way you did above. Give...


Near line 748: Possible spelling mistake found.
Context: ..., it will increment the guess count for wordle-key's account. We can verify this by query...


Near line 756: Only proper nouns start with an uppercase character (there are exceptions for headlines).
Context: ...ess --output json ``` This outputs all Guess objects submitted so far, with the inde...


Near line 760: Possible spelling mistake found.
Context: ... Wordle using Cosmos-SDK and Ignite and Rollkit. Read on to how you can extend the code...


Near line 771: ‘prior to’ might be wordy. Consider a shorter alternative.
Context: ... correct word. 2. You can hash the word prior to submitting it to the chain, ensuring ...


Near line 777: You might be missing the article “the” here.
Context: ...mblebun/wordle-cli). 4. You can improve current date to stick to a specific timezone. 5...


Near line 778: Possible spelling mistake found.
Context: ... 5. You can create a bot that submits a wordle every day at a specific time. 6. You ca...


Near line 779: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...wordle every day at a specific time. 6. You can create a vue.js front-end with Igni...

Additional comments not posted (6)
tutorials/wordle.md (6)

334-334: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-    return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle Must Be A 5 Letter Word")
+    return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle Must Be A 5 Letter Word")

338-338: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-    return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle Must Only Consist Of Letters In The Alphabet")
+    return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle Must Only Consist Of Letters In The Alphabet")

360-360: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-    return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle of the Day is Already Submitted")
+    return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "Wordle of the Day is Already Submitted")

429-429: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-    return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "Guess Must Be A 5 Letter Word!")
+    return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "Guess Must Be A 5 Letter Word!")

434-434: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-    return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "Guess Must Only Consist of Alphabet Letters!")
+    return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "Guess Must Only Consist of Alphabet Letters!")

462-462: Use sdkerrors.Wrap directly for error handling instead of errors.Wrap.

-      return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "You Have Guessed The Maximum Amount of Times for The Day! Try Again Tomorrow With A New Wordle.")
+      return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "You Have Guessed The Maximum Amount of Times for The Day! Try Again Tomorrow With A New Wordle.")

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?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

github-actions bot commented Apr 18, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-04-23 13:55 UTC

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 9

Actionable comments outside the diff hunks (4)
tutorials/wordle.md (4)

Line range hint 9-9: Correct the spelling of "setup" to "set up" when used as a verb.

- This tutorial will go over how to setup Rollkit
+ This tutorial will go over how to set up Rollkit

Line range hint 17-17: Capitalize "GitHub" in the text.

- please write a Github
+ please write a GitHub

Line range hint 98-98: Correct the spelling of "MacOS" to "macOS".

- This tutorial uses a MacOS but it should work for Windows.
+ This tutorial uses a macOS but it should work for Windows.

Line range hint 129-129: Correct the spelling of "setup" to "set up" when used as a verb.

- We first run the following command to setup our project
+ We first run the following command to set up our project

tutorials/wordle.md Outdated Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
tutorials/wordle.md Outdated Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
tutorials/wordle.md Show resolved Hide resolved
scripts/gm/init-arabica-testnet.sh Show resolved Hide resolved
Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

Actionable comments outside the diff hunks (1)
tutorials/wordle.md (1)

Line range hint 9-9: Correct the spelling of "setup" to "set up" when used as a verb.

- setup Rollkit
+ set up Rollkit

@gupadhyaya gupadhyaya enabled auto-merge April 23, 2024 13:55
@gupadhyaya gupadhyaya merged commit 7c3b9b5 into main Apr 23, 2024
4 checks passed
@gupadhyaya gupadhyaya deleted the yarik/tutorial-fixes branch April 23, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants