-
Notifications
You must be signed in to change notification settings - Fork 365
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
docs: migrate node ID in bridge-node, full-storage-node, light-node #1685
Conversation
Caution Review failedThe pull request is closed. WalkthroughThe changes introduce new sections in the documentation for the Celestia bridge, full storage, and light nodes. Each section outlines a two-step process for migrating a node ID to a new server, which includes backing up specific files and uploading them to the new server before starting the node. This addition provides users with clear instructions for node ID migration. 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 as PR comments)
Additionally, you can add CodeRabbit Configuration 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: 3
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- nodes/bridge-node.md (1 hunks)
- nodes/full-storage-node.md (1 hunks)
- nodes/light-node.md (1 hunks)
Additional context used
LanguageTool
nodes/full-storage-node.md
[grammar] ~155-~155: The word “backup” is a noun. The verb is spelled with a space.
Context: ...a full storage node ID: 1. You need to backup two files located in the celestia-full ...(NOUN_VERB_CONFUSION)
nodes/light-node.md
[grammar] ~198-~198: The word “backup” is a noun. The verb is spelled with a space.
Context: ...igrate a light node ID: 1. You need to backup two files located in the celestia-light...(NOUN_VERB_CONFUSION)
nodes/bridge-node.md
[grammar] ~196-~196: The word “backup” is a noun. The verb is spelled with a space.
Context: ...grate a bridge node ID: 1. You need to backup two files located in the celestia-bridg...(NOUN_VERB_CONFUSION)
Additional comments not posted (3)
nodes/full-storage-node.md (1)
151-157
: Approve new section on node ID migration.The new section provides clear, step-by-step instructions for migrating a node ID, which enhances the documentation and supports the PR's objectives.
Tools
LanguageTool
[grammar] ~155-~155: The word “backup” is a noun. The verb is spelled with a space.
Context: ...a full storage node ID: 1. You need to backup two files located in the celestia-full ...(NOUN_VERB_CONFUSION)
nodes/light-node.md (1)
194-200
: Approve new section on light node ID migration.The new section provides clear, step-by-step instructions for migrating a light node ID, which enhances the documentation and supports the PR's objectives.
Tools
LanguageTool
[grammar] ~198-~198: The word “backup” is a noun. The verb is spelled with a space.
Context: ...igrate a light node ID: 1. You need to backup two files located in the celestia-light...(NOUN_VERB_CONFUSION)
nodes/bridge-node.md (1)
192-198
: Approve new section on bridge node ID migration.The new section provides clear, step-by-step instructions for migrating a bridge node ID, which enhances the documentation and supports the PR's objectives.
Tools
LanguageTool
[grammar] ~196-~196: The word “backup” is a noun. The verb is spelled with a space.
Context: ...grate a bridge node ID: 1. You need to backup two files located in the celestia-bridg...(NOUN_VERB_CONFUSION)
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
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.
thank you @doodleoo3 !
Overview
Currently, the node id migration process is not very transparent in the documentation. This PR fixes that
Summary by CodeRabbit