-
Notifications
You must be signed in to change notification settings - Fork 178
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
update devnet #834
update devnet #834
Conversation
WalkthroughThe recent changes update the installation and setup instructions for the development environment. Key modifications include the addition of the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CLI
participant Go
participant Foundry
User->>CLI: Install just
CLI-->>User: Download and install just
User->>CLI: Set up Go
CLI-->>Go: Install Go
Go-->>CLI: Configure PATH
User->>CLI: Install Foundry
CLI-->>Foundry: Execute just install-foundry
Foundry-->>CLI: Installation status
Recent review detailsConfiguration used: .coderabbit.yaml Files selected for processing (1)
Additional context usedPath-based instructions (1)
LanguageTool
Additional comments not posted (6)
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 Configuration File (
|
✅ Deploy Preview for docs-optimism ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This might be a matter of style preference. I made these changes simply to update the document, which is currently outdated. I updated it in a way that continues the original style. Since I'm not a native English speaker, I'm not skilled at making stylistic changes to the text |
Thanks for the contribution, someone will check it soon. |
Additional:Installation methods are also mentioned in other locations: https://github.com/ethereum-optimism/superchain-ops . |
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
Outside diff range, codebase verification and nitpick comments (1)
pages/chain/testing/dev-node.mdx (1)
91-106
: Improve sentence variety for clarity.The repetitive sentence structure can be improved for better readability. Consider rephrasing the sentences to avoid starting multiple sentences with "If you."
Apply this diff to improve the sentence structure:
If you do already have Foundry installed, make sure to run the following to verify your installed version will work with the monorepo tooling as expected. ```sh just check-foundry
- If you need to, you can run the following to update Foundry to the expected version.
- To update Foundry to the expected version, run the following command if necessary.
<details> <summary>Tools</summary> <details> <summary>LanguageTool</summary><blockquote> [style] ~103-~103: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym. Context: ... ```sh just check-foundry ``` If you need to, you can run the following ... (ENGLISH_WORD_REPEAT_BEGINNING_RULE) </blockquote></details> </details> </blockquote></details> </blockquote></details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
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: 4
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
Outside diff range, codebase verification and nitpick comments (1)
pages/chain/testing/dev-node.mdx (1)
100-100
: Avoid repetitive sentence structure.Consider rephrasing to avoid starting three successive sentences with "If you." This will enhance readability and engagement.
Here's a suggestion:
```sh just check-foundry
- If you need to, you can run the following to update Foundry to the expected version.
- To update Foundry to the expected version, run the following command if necessary.
<details> <summary>Tools</summary> <details> <summary>LanguageTool</summary><blockquote> [style] ~100-~100: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym. Context: ... ```sh just check-foundry ``` If you need to, you can run the following ... (ENGLISH_WORD_REPEAT_BEGINNING_RULE) </blockquote></details> </details> </blockquote></details> </blockquote></details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
Description
This update is in response to changes on the develop branch.
Tests
Tested:
op-challenger logs:
Additional context
Add any other context about the problem you're solving.
Metadata