-
Notifications
You must be signed in to change notification settings - Fork 357
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: add ginger upgrade to docs, update links #1758
Conversation
WalkthroughThe pull request introduces several updates across multiple documentation files, primarily focusing on clarifying and enhancing content related to the Arabica devnet, Mainnet Beta, Mocha testnet, and the network upgrade process. Key changes include the transition from referencing the "Lemongrass network upgrade" to the "Ginger network upgrade," improved readability, and additional details about network stability, RPC endpoints, and the upgrade coordination process. Changes
Possibly related PRs
Suggested labels
Suggested reviewers
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 using PR comments)
Other keywords and placeholders
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: 0
🧹 Outside diff range and nitpick comments (8)
how-to-guides/network-upgrade-process.md (3)
7-7
: Minor improvements for clarity and concisenessThe introduction paragraph provides a good overview of network upgrades. Consider the following minor adjustments to improve clarity and address some language issues:
- Add a comma after "new features" for better readability.
- Consider replacing "prior to" with "before" for conciseness.
- Add a comma after "network upgrades" in the last sentence.
Here's a suggested revision:
Blockchain networks often need to upgrade with new features, which require coordination work among the community of developers, validators, and node operators before activating the upgrades. This process is called a network upgrade, which can be breaking or non-breaking. During planned network upgrades, the community will coordinate to prepare for the upcoming upgrade. Breaking network upgrades are not backward-compatible with older versions of the network software, which is why it is important that validators upgrade their software to continue validating on the network. Non-breaking network upgrades are backward-compatible and require less coordination.🧰 Tools
🪛 LanguageTool
[uncategorized] ~7-~7: Possible missing comma found.
Context: ...networks often need to upgrade with new features which require coordination work among t...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~7-~7: ‘prior to’ might be wordy. Consider a shorter alternative.
Context: ...elopers, validators, and node operators prior to activating the upgrades. This process i...(EN_WORDINESS_PREMIUM_PRIOR_TO)
[uncategorized] ~7-~7: Possible missing comma found.
Context: ...or non-breaking. During planned network upgrades the community will coordinate to prepar...(AI_HYDRA_LEO_MISSING_COMMA)
28-45
: Comprehensive overview of the upcoming Ginger upgradeThis new section provides an excellent overview of the upcoming Ginger network upgrade. The content is well-structured and informative, covering key features, relevant CIPs, and important information about the in-protocol signaling mechanism.
The info box for validators is a crucial reminder and is well-placed within the section.
To further enhance this section, consider adding a brief explanation or link to more information about each CIP mentioned in the list of key features. This would help readers quickly understand the purpose of each CIP without having to navigate away from the document.
55-56
: Informative overview of the Lemongrass network upgradeThis subsection provides valuable historical context about the Lemongrass network upgrade. The information about it being the first consensus layer breaking change and the reference to relevant CIPs is important for understanding the upgrade's significance.
Consider a minor language improvement:
- The Lemongrass network upgrade included all of the CIPs listed in [CIP-17](https://github.com/celestiaorg/CIPs/blob/main/cip-17.md) and implemented CIP-10 for future upgrades. + The Lemongrass network upgrade included the CIPs listed in [CIP-17](https://github.com/celestiaorg/CIPs/blob/main/cip-17.md) and implemented CIP-10 for future upgrades.This change makes the sentence slightly more concise without altering its meaning.
🧰 Tools
🪛 LanguageTool
[style] ~55-~55: Consider removing “of” to be more concise
Context: ...The Lemongrass network upgrade included all of the CIPs listed in [CIP-17](https://github....(ALL_OF_THE)
how-to-guides/mocha-testnet.md (1)
246-246
: LGTM! Consider adding the upgrade date for context.The update to reference the Ginger network upgrade is appropriate and consistent with changes in other documents. The added link to the specific section about the Ginger upgrade is helpful for users.
Consider adding the date of the Ginger upgrade in parentheses for additional context. For example:
about specific upgrades like the [Ginger network upgrade](./network-upgrade-process.md#ginger-network-upgrade) (Date: Month Day, Year).how-to-guides/arabica-devnet.md (3)
184-186
: LGTM! Consider adding a brief description of the Ginger upgrade.The changes look good. The addition of the Telegram announcement channel is helpful for users to stay informed about network upgrades. The update from "Lemongrass" to "Ginger" network upgrade is consistent with changes mentioned in the AI-generated summary.
Consider adding a brief description of what the Ginger network upgrade entails, either inline or by updating the linked network upgrade process page. This would provide users with immediate context about the upgrade's significance.
Line range hint
46-103
: Great additions! Consider adding a note about endpoint selection.The changes in this section are excellent. The addition of production RPC endpoints and the warning about community endpoints provide valuable guidance for users. The reformatted table of community RPC endpoints is now more comprehensive and easier to read.
Consider adding a brief note explaining how users should choose between production and community endpoints based on their use case. This could help users make informed decisions about which endpoints to use.
Line range hint
105-145
: Excellent additions! Consider clarifying the difference between RPC and gRPC endpoints.The new information about DA RPC and gRPC endpoints is very helpful. The example command for starting a light node and the tip for bridge node runners provide valuable practical guidance.
Consider adding a brief explanation of the difference between RPC and gRPC endpoints and when a user might choose one over the other. This could help users better understand which endpoint type they need for their specific use case.
how-to-guides/mainnet.md (1)
306-306
: LGTM! Consider adding a brief description of the Ginger upgrade.The addition of the Ginger network upgrade information is valuable and aligns with the PR objectives. It provides users with a specific reference to learn more about the latest upgrade.
Consider adding a brief (1-2 sentence) description of what the Ginger upgrade entails, directly in this document. This would give readers immediate context without requiring them to follow the link.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (5)
- how-to-guides/arabica-devnet.md (1 hunks)
- how-to-guides/mainnet.md (1 hunks)
- how-to-guides/mocha-testnet.md (1 hunks)
- how-to-guides/network-upgrade-process.md (1 hunks)
- how-to-guides/participate.md (1 hunks)
🧰 Additional context used
🪛 LanguageTool
how-to-guides/network-upgrade-process.md
[uncategorized] ~7-~7: Possible missing comma found.
Context: ...networks often need to upgrade with new features which require coordination work among t...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~7-~7: ‘prior to’ might be wordy. Consider a shorter alternative.
Context: ...elopers, validators, and node operators prior to activating the upgrades. This process i...(EN_WORDINESS_PREMIUM_PRIOR_TO)
[uncategorized] ~7-~7: Possible missing comma found.
Context: ...or non-breaking. During planned network upgrades the community will coordinate to prepar...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~55-~55: Consider removing “of” to be more concise
Context: ...The Lemongrass network upgrade included all of the CIPs listed in [CIP-17](https://github....(ALL_OF_THE)
🔇 Additional comments (6)
how-to-guides/participate.md (1)
64-64
: LGTM: Update to Ginger network upgrade reference.The change from "Lemongrass network upgrade" to "Ginger network upgrade" is consistent with the PR objectives and aligns with updates made in other files.
Let's verify that the linked page contains information about the Ginger network upgrade:
✅ Verification successful
: The linked page contains information about the Ginger network upgrade.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that the network-upgrade-process.md file contains information about the Ginger upgrade. # Test: Search for "Ginger network upgrade" in the network-upgrade-process.md file rg -i "ginger network upgrade" how-to-guides/network-upgrade-process.mdLength of output: 348
how-to-guides/network-upgrade-process.md (5)
9-16
: Excellent addition of network upgrade coordination detailsThis new section provides valuable information about the network upgrade coordination process. It clearly explains the two methods for coordinating upgrades and introduces the in-protocol signaling mechanism. The inclusion of the quorum requirement (5/6 of validators) is particularly important.
The content is well-structured and easy to understand. Great job on this addition!
18-26
: Well-structured and informative upgrade process sectionThe renaming and restructuring of this section significantly improve its clarity and readability. The numbered list of steps provides a clear overview of the upgrade process, making it easier for readers to understand the sequence of events.
The inclusion of information about both pre-programmed height and in-protocol signaling methods is valuable, as it covers both the current and future upgrade approaches.
Overall, these changes enhance the document's usefulness for the Celestia community. Great work on improving this section!
47-49
: Important warning for validatorsThe addition of this warning box is crucial for ensuring a smooth upgrade process. It clearly advises validators against using tools like cosmovisor and emphasizes the importance of upgrading the binary before signaling support for the new version.
This warning helps prevent potential issues during the upgrade process and is a valuable addition to the document.
51-53
: Improved document structure with Past Upgrades sectionThe addition of the "Past Upgrades" section header enhances the overall structure of the document. It clearly separates information about past upgrades from upcoming ones, making it easier for readers to navigate the content and find relevant information.
This structural improvement contributes to better organization and readability of the document.
57-61
: Clear and informative upgrade height tableThe addition of this table is excellent. It provides a concise and easy-to-read summary of the Lemongrass upgrade details across different networks. The inclusion of chain IDs, dates, times, and upgrade heights for Arabica, Mocha, and Mainnet Beta is particularly useful for users and operators across various Celestia networks.
This table enhances the document's value as a reference for the Lemongrass upgrade information.
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.
LGTM with a few optional nits.
[optional] we could remove the Network upgrades section from arabica, mocha, mainnet beta pages b/c it is duplicative of the network upgrade process page.
Co-authored-by: Rootul P <rootulp@gmail.com>
this is a good suggestion, I think I'll leave it in for discoverability, for now |
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 and nitpick comments (3)
how-to-guides/network-upgrade-process.md (3)
7-7
: Consider more concise wordingThe sentence could be more concise by replacing "prior to" with "before".
-Blockchain networks often need to upgrade with new features which require coordination work among the community of developers, validators, and node operators prior to activating the upgrades. +Blockchain networks often need to upgrade with new features which require coordination work among the community of developers, validators, and node operators before activating the upgrades.🧰 Tools
🪛 LanguageTool
[style] ~7-~7: ‘prior to’ might be wordy. Consider a shorter alternative.
Context: ...elopers, validators, and node operators prior to activating the upgrades. This process i...(EN_WORDINESS_PREMIUM_PRIOR_TO)
34-39
: Consider adding version numbers for CIPsFor better traceability, consider adding version numbers or status indicators for each CIP. This helps readers understand which version of each CIP is being implemented.
55-55
: Consider more concise wordingThe sentence could be more concise by removing "all of".
-The Lemongrass network upgrade (v2) was the first consensus layer breaking change since Celestia's Mainnet Beta genesis block. The Lemongrass network upgrade included all of the CIPs listed in [CIP-17] +The Lemongrass network upgrade (v2) was the first consensus layer breaking change since Celestia's Mainnet Beta genesis block. The Lemongrass network upgrade included the CIPs listed in [CIP-17]🧰 Tools
🪛 LanguageTool
[style] ~55-~55: Consider removing “of” to be more concise
Context: ...The Lemongrass network upgrade included all of the CIPs listed in [CIP-17](https://github....(ALL_OF_THE)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- how-to-guides/network-upgrade-process.md (1 hunks)
🧰 Additional context used
🪛 LanguageTool
how-to-guides/network-upgrade-process.md
[style] ~7-~7: ‘prior to’ might be wordy. Consider a shorter alternative.
Context: ...elopers, validators, and node operators prior to activating the upgrades. This process i...(EN_WORDINESS_PREMIUM_PRIOR_TO)
[style] ~55-~55: Consider removing “of” to be more concise
Context: ...The Lemongrass network upgrade included all of the CIPs listed in [CIP-17](https://github....(ALL_OF_THE)
🔇 Additional comments (3)
how-to-guides/network-upgrade-process.md (3)
9-16
: Well-structured explanation of upgrade coordination methodsThe section clearly explains both upgrade coordination methods and the in-protocol signaling mechanism. The reference to CIP-10 and the explanation of the 5/6 quorum requirement are particularly helpful.
18-26
: LGTM! Clear and comprehensive upgrade process stepsThe step-by-step process is well-documented and includes both pre-programmed height and in-protocol signaling mechanisms.
34-39
: Verify CIP links are accessibleLet's verify that all referenced CIP links are accessible and point to the correct documents.
✅ Verification successful
Let me verify the actual content and format of the CIP links in the document to ensure they are correctly formatted and consistent.
All CIP links are properly formatted and consistent
All CIP links in the document follow the correct format
https://cips.celestia.org/cip-{number}.html
, with the only exception being CIP-17 which appropriately points to the GitHub repository. The links for all CIPs mentioned in the upgrade features (CIP-21, CIP-24, CIP-26, CIP-27, and CIP-28) are consistently formatted and present in the document.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that all CIP links are accessible # Test: Check if all referenced CIPs exist in the repository echo "Checking CIP links..." for cip in 10 21 24 25 26 27 28; do rg -l "cip-${cip}" || echo "Warning: No references found for CIP-${cip}" doneLength of output: 682
Script:
#!/bin/bash # Check the actual format and content of CIP links in the document rg -U "CIP-\d+" how-to-guides/network-upgrade-process.md -A 1 -B 1Length of output: 1262
Overview
Summary by CodeRabbit