-
Notifications
You must be signed in to change notification settings - Fork 305
Fix typos and grammatical errors in node-operators documentation #1845
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
Conversation
- Fix capitalization: 'Your' -> 'your' in heading - Add missing article 'the' in genesis URL placeholder - Remove redundant 'use' in description - Fix possessive form: 'OP Labs's' -> 'OP Labs' - Remove duplicate text about beacon-archiver configuration - Remove redundant 'download' before 'access' - Add missing 'will' before 'log' - Add missing article 'a' before 'database'
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
| --l1.beacon-archiver value ($OP_NODE_L1_BEACON) | ||
| HTTP Endpoint of a Blob Archiver or an L1 Beacon-node that does not prune blobs | ||
| ``` | ||
| Running a `blob-archiver` is lighter weight than running a beacon node that does not prune blobs: [https://github.com/base-org/blob-archiver](https://github.com/base-org/blob-archiver). |
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.
Devin, can you revert these changes?
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.
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.
oh okay, are you sure the one with the @ is correct? that seems like a random github user
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.
devinai, can you revert these changes?
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.
yes, it's kind of weird but I'm sure that with the @ works
- Restore beacon-archiver documentation in blobs.mdx (lines 102-109) - Fix placeholder text in base-config.mdx: 'op-geth the url' -> 'the op-geth URL' - Fix grammar in regenesis-history.mdx: 'data recover' -> 'data recovery'
Fix typos and grammatical errors in node-operators documentation
Summary
Fixed 7 grammatical errors and typos across 4 documentation files in the node-operators folder. Changes are strictly grammar and spelling corrections - no content restructuring or technical changes.
Files changed:
node-operators/guides/base-config.mdx- Fixed heading capitalization and improved placeholder text grammarnode-operators/guides/management/regenesis-history.mdx- Fixed description, possessive form, and noun formnode-operators/guides/management/snapshots.mdx- Removed redundant wordnode-operators/tutorials/run-node-from-source.mdx- Added missing verb and articleReview & Testing Checklist for Human
base-config.mdxline 56 reads naturally:<<insert the op-geth URL to the genesis file>>- does "the op-geth URL" make sense in this context, or should it be different?base-config.mdxline 17 ("Your" → "your") aligns with the repo's style guide - this could be a style preference rather than a strict errorNotes
blobs.mdx(reverted in follow-up)base-config.mdxplaceholder text (corrected in follow-up)Last update: 2025-11-05 15:35 ET