-
Notifications
You must be signed in to change notification settings - Fork 107
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
added chiado updates #890
base: main
Are you sure you want to change the base?
added chiado updates #890
Conversation
Warning Rate limit exceeded@ijonele has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 15 minutes and 47 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 Walkthrough📝 Walkthrough📝 WalkthroughWalkthroughThe pull request introduces several updates to the documentation, primarily focusing on the transition from the Buenavista testnet to the Chiado testnet. Key changes include the addition of new entries in the spelling file, updates to various documents to reflect the new testnet, and modifications to URLs and terminology. The changes ensure that users are directed to the correct resources and instructions for operating within the Chiado testnet environment. Changes
Possibly related PRs
Suggested reviewers
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
Documentation and Community
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
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
Outside diff range and nitpick comments (11)
docs/developer-docs/docs/operate-a-node/buenavista-testnet/_category_.json (2)
5-5
: Consider the impact of collapsing the Buenavista testnet section by default.Changing the
collapsed
state totrue
will hide the Buenavista testnet content by default. While this can help reduce clutter in the documentation, it may also make it less discoverable for users specifically looking for this information.Consider the following:
- Is the Buenavista testnet information accessed frequently enough to warrant being expanded by default?
- Are there other sections in the documentation that follow a similar collapsed/expanded pattern?
- How does this change align with the overall documentation structure and user experience goals?
It might be helpful to gather feedback from users or conduct A/B testing to determine the optimal default state for this section.
2-5
: Verify changes in the context of overall documentation structure.The changes to both the position and collapsed state of the Buenavista testnet section could significantly impact how users navigate and interact with the documentation. To ensure these changes enhance the overall user experience:
- Review the entire documentation structure to confirm that the new position (4) of this section maintains a logical flow of information.
- Check if other sections have similar collapsed states to maintain consistency throughout the documentation.
- Consider adding a brief note or tooltip explaining why this section is collapsed by default, if it contains information that's less frequently accessed or intended for a specific subset of users.
- If possible, gather feedback from documentation users to validate that these changes improve their experience in finding and accessing information about the Buenavista testnet.
docs/developer-docs/docs/operate-a-node/chiado-testnet/_category_.json (1)
1-11
: LGTM with minor suggestionsThe JSON configuration for the Chiado testnet category looks good overall. Here are a few suggestions for improvement:
Verify that the "position" value of 3 is correct within the overall documentation structure.
Consider removing the empty "customProps" object if there are no immediate plans to use it. This can help keep the configuration concise.
For consistency with typical network naming conventions, consider capitalizing "testnet" in the label and title:
- "label": "Chiado testnet", + "label": "Chiado Testnet", ... "link": { "type": "generated-index", - "title": "Chiado testnet" + "title": "Chiado Testnet" },docs/developer-docs/docs/operate-a-node/introduction.md (1)
Line range hint
1-33
: Minor improvements for document structure and clarity.The document is well-written and provides clear instructions. However, there are a couple of minor points that could be improved:
- There's a numbering inconsistency in the steps: there are two step 4's. Please correct the numbering.
- For improved clarity, consider adding a brief explanation of what Chiado is when it's first mentioned, as new users might not be familiar with it.
Here's a suggested change for point 2:
-2. To get started with our testnet, follow [Join Chiado](chiado-testnet/join-chiado). +2. To get started with our testnet (Chiado), follow [Join Chiado](chiado-testnet/join-chiado).docs/developer-docs/.spelling (1)
253-254
: LGTM! The additions are relevant and consistent.The new entries "Evmos" and "Chiado" are appropriate additions to the spelling configuration. They are consistent with the existing blockchain-related terminology in the file.
Consider adding related terms if they're frequently used in your documentation, such as:
- "Gnosis" (the blockchain associated with Chiado testnet)
- "xDai" (former name of Gnosis Chain)
- "EVM" (if not already included, as Evmos is EVM-compatible)
This will ensure comprehensive coverage of related terminology in your spell-checking process.
docs/developer-docs/docs/operate-a-node/node-commands.md (2)
31-31
: LGTM: Node URL updateThe node URL has been correctly updated to
https://rpc.chiado.wardenprotocol.org:443
, which is consistent with the migration from Buenavista to Chiado.Consider adding a brief comment or note explaining the change from Buenavista to Chiado for users who might be familiar with the old URL. For example:
# Note: The node URL has been updated to use the Chiado network instead of Buenavista. wardend status \ --node https://rpc.chiado.wardenprotocol.org:443
146-147
: LGTM: Chain ID and node URL updates in transaction examplesThe chain ID has been correctly updated to
chiado_10010-1
, and the node URL tohttps://rpc.chiado.wardenprotocol.org:443
in both transaction examples. These changes are consistent with the migration from Buenavista to Chiado.Consider adding a brief note explaining the new chain ID format, as it differs significantly from the previous one. For example:
# Note: The chain ID for Chiado follows a new format: network-name_network-id-1 --chain-id chiado_10010-1 \This will help users understand the structure of the new chain ID and prevent confusion.
Also applies to: 155-156
docs/developer-docs/docs/operate-a-node/buenavista-testnet/join-buenavista.md (1)
18-20
: Approve the addition of the warning message with a minor suggestion.The warning message about transitioning from Buenavista to Chiado testnet is clear and informative. It effectively communicates the current status and the need for users to migrate their processes.
Consider replacing "we're" with "we are" for a more formal tone:
-We're transitioning from Buenavista to our new and improved testnet, [Chiado](../chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we're going to sunset Buenavista. Please make sure to transition all your testing and development processes to [Chiado](../chiado-testnet/join-chiado). +We are transitioning from Buenavista to our new and improved testnet, [Chiado](../chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we are going to sunset Buenavista. Please make sure to transition all your testing and development processes to [Chiado](../chiado-testnet/join-chiado).Tools
LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...sta. Please make sure to transition all your testing and development processes to [C...(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...Chiado](../chiado-testnet/join-chiado). ::: ## Version history | Release | Upgr...(UNLIKELY_OPENING_PUNCTUATION)
docs/developer-docs/docs/learn/glossary.md (3)
141-141
: LGTM with a minor suggestion for clarityThe update from "uWARD" to "aWARD" is correct and consistent. The new value (0.000000000000000001 WARD) is accurately defined.
To enhance clarity, consider adding a brief explanation of why this change was made or how it affects users.
226-226
: LGTM with a suggestion for consistencyThe update from "uWARD" to "aWARD" is correct and consistent with the previous change. The new value (0.000000000000000001 WARD) is accurately defined.
For consistency with the key request fee section, consider adding a brief explanation of why this change was made or how it affects users.
Line range hint
1-365
: Suggestion for improving consistency across similar entriesThe glossary is well-structured and comprehensive. To further enhance its usefulness, consider standardizing the information provided for similar types of entries. For example, for fee-related entries (like "Key request fee" and "Signature request fee"), you could include:
- A brief explanation of the purpose of the fee
- Who sets the fee
- How the fee is collected and distributed
- Any limitations or restrictions on fee amounts
This standardization would make it easier for readers to compare and understand related concepts.
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (16)
- docs/developer-docs/.spelling (1 hunks)
- docs/developer-docs/docs/build-a-keychain/operate-a-keychain/create-a-keychain.md (2 hunks)
- docs/developer-docs/docs/build-a-keychain/operate-a-keychain/fulfill-requests-from-cli.md (1 hunks)
- docs/developer-docs/docs/build-an-app/deploy-smart-contracts-on-warden/deploy-a-cross-chain-app.md (1 hunks)
- docs/developer-docs/docs/build-an-app/examples-of-oapps.md (1 hunks)
- docs/developer-docs/docs/learn/glossary.md (2 hunks)
- docs/developer-docs/docs/operate-a-node/buenavista-testnet/category.json (1 hunks)
- docs/developer-docs/docs/operate-a-node/buenavista-testnet/join-buenavista.md (2 hunks)
- docs/developer-docs/docs/operate-a-node/chiado-testnet/category.json (1 hunks)
- docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1 hunks)
- docs/developer-docs/docs/operate-a-node/create-a-validator.md (4 hunks)
- docs/developer-docs/docs/operate-a-node/delegation-plan.md (2 hunks)
- docs/developer-docs/docs/operate-a-node/introduction.md (1 hunks)
- docs/developer-docs/docs/operate-a-node/node-api-reference.md (1 hunks)
- docs/developer-docs/docs/operate-a-node/node-commands.md (4 hunks)
- docs/developer-docs/docs/operate-a-node/operate-skip-connect.md (1 hunks)
Files skipped from review due to trivial changes (4)
- docs/developer-docs/docs/build-an-app/deploy-smart-contracts-on-warden/deploy-a-cross-chain-app.md
- docs/developer-docs/docs/build-an-app/examples-of-oapps.md
- docs/developer-docs/docs/operate-a-node/delegation-plan.md
- docs/developer-docs/docs/operate-a-node/node-api-reference.md
Additional context used
Path-based instructions (9)
docs/developer-docs/docs/build-a-keychain/operate-a-keychain/create-a-keychain.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/build-a-keychain/operate-a-keychain/fulfill-requests-from-cli.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/learn/glossary.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/buenavista-testnet/join-buenavista.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/create-a-validator.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/introduction.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/node-commands.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/operate-skip-connect.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
LanguageTool
docs/developer-docs/docs/operate-a-node/buenavista-testnet/join-buenavista.md
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...sta. Please make sure to transition all your testing and development processes to [C...(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...Chiado](../chiado-testnet/join-chiado). ::: ## Version history | Release | Upgr...(UNLIKELY_OPENING_PUNCTUATION)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...should use a new denomination,award
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
Markdownlint
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
82-82: null
Bare URL used(MD034, no-bare-urls)
Additional comments not posted (25)
docs/developer-docs/docs/operate-a-node/buenavista-testnet/_category_.json (1)
2-2
: Verify the intended position of the Buenavista testnet section.The position of this section has been changed from 3 to 4. Please confirm that this new position aligns with the intended documentation structure and maintains a logical flow of information for users.
docs/developer-docs/docs/operate-a-node/operate-skip-connect.md (3)
2-2
: Verify the sidebar position change.The sidebar position has been changed from 5 to 6. Please confirm if this change is intentional and doesn't disrupt the logical flow of the documentation in the sidebar.
Line range hint
1-67
: Documentation content looks good.The content provides clear and concise instructions for operating Skip:Connect. The information appears to be complete, correct, and well-structured. No misspellings or grammatical errors were found.
Line range hint
67-67
: Verify the removal of the "What's next?" section.The "What's next?" section, which previously contained information about Connect availability in Warden version v0.4.0, has been removed. Please confirm if this removal is intentional and that no crucial information has been lost. If the information is still relevant, consider adding it to an appropriate section or creating a new section for future steps or version compatibility.
docs/developer-docs/docs/operate-a-node/node-commands.md (4)
2-2
: LGTM: Sidebar position updateThe change in sidebar position from 8 to 9 has been noted. This appears to be an intentional modification, likely part of a broader documentation restructuring.
78-78
: LGTM: Consistent node URL updateThe node URL has been correctly updated to
https://rpc.chiado.wardenprotocol.org:443
in the keychain query example, maintaining consistency with the previous updates.
85-85
: LGTM: Consistent node URL update in spaces queryThe node URL has been correctly updated to
https://rpc.chiado.wardenprotocol.org:443
in the spaces query example, maintaining consistency across all examples in the document.
Line range hint
1-156
: Overall assessment: Documentation successfully updated for Chiado migrationThe document has been thoroughly updated to reflect the migration from Buenavista to Chiado. All node URLs and chain IDs have been consistently changed throughout the examples. The modifications improve the accuracy and relevance of the documentation for users interacting with the Chiado network.
Key changes:
- Updated node URLs to
https://rpc.chiado.wardenprotocol.org:443
- Updated chain ID to
chiado_10010-1
- Adjusted sidebar position
These updates ensure that users have the correct information to interact with the Warden node on the Chiado network. The document maintains its clarity and structure while incorporating these necessary changes.
docs/developer-docs/docs/operate-a-node/create-a-validator.md (5)
2-2
: LGTM: Updated testnet referenceThe change from Buenavista to Chiado testnet in the introductory text is correct and consistent with the PR objectives.
Also applies to: 7-7
11-11
: LGTM: Updated link text, please verify destinationThe change to reference Chiado testnet join instructions is correct. However, please verify that the linked page (chiado-testnet/join-chiado) exists and contains the correct information for joining the Chiado testnet.
44-44
: LGTM: Updated faucet URL, please verify functionalityThe faucet URL has been correctly updated to reflect the Chiado testnet. However, it's crucial to ensure that this URL is functional and correctly configured for the Chiado testnet.
Please verify that the faucet URL (https://faucet.chiado.wardenprotocol.org) is operational and correctly distributes Chiado testnet tokens.
76-76
: Please clarify and verify the token amount and denomination changeThe validator creation amount has been significantly increased from "1000000uward" to "1000000000000000000award". This change also involves a shift in denomination from "uward" to "award".
Could you please clarify:
- Is this increase intentional and correct for the Chiado testnet?
- Does the change from "uward" to "award" reflect a new token denomination system?
- How does this amount translate to WARD tokens in the Chiado testnet?
Please confirm that this new amount and denomination are correct for creating a validator on the Chiado testnet.
96-97
: LGTM: Updated chain-id, please verify fee amountThe chain-id has been correctly updated to "chiado_10010-1", which is consistent with the transition to the Chiado testnet.
However, the fee amount has been significantly increased from "500uward" to "500000000000000award". This change is similar to the validator creation amount change observed earlier.
Please confirm that:
- The new chain-id "chiado_10010-1" is correct for the Chiado testnet.
- The new fee amount "500000000000000award" is correct and not excessive for creating a validator on the Chiado testnet.
docs/developer-docs/docs/operate-a-node/buenavista-testnet/join-buenavista.md (3)
104-104
: Approve the formatting improvement in the export command.The removal of trailing spaces in the
SNAP_RPC_SERVERS
export command enhances readability without affecting functionality.
19-19
: Disregard false positives from static analysis.The static analysis tool has flagged two issues that are actually false positives:
- The suggestion to replace "your" with "you're" is incorrect. "Your" is the correct possessive form in "your testing and development processes".
- The flagged loose punctuation mark at the end of the line is part of the Markdown syntax for closing a warning block and is not an error.
No changes are needed based on these static analysis hints.
Tools
LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...sta. Please make sure to transition all your testing and development processes to [C...(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...Chiado](../chiado-testnet/join-chiado). ::: ## Version history | Release | Upgr...(UNLIKELY_OPENING_PUNCTUATION)
Line range hint
1-138
: Approve the overall state of the documentation.The documentation for joining the Buenavista testnet is well-structured, clear, and up-to-date. It includes:
- The latest version information (v0.4.2)
- Proper links to external resources
- Clear instructions for installation, configuration, and node startup
- Information about state sync and next steps
No significant issues or omissions were found in the rest of the document.
Tools
LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...sta. Please make sure to transition all your testing and development processes to [C...(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...Chiado](../chiado-testnet/join-chiado). ::: ## Version history | Release | Upgr...(UNLIKELY_OPENING_PUNCTUATION)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (6)
1-6
: LGTM: Frontmatter and import statement are correct.The frontmatter is properly formatted, and the import statement for the
PersistentPeers
component is correctly included.
7-20
: LGTM: Overview section is informative and well-structured.The overview provides essential information about the Chiado testnet, including chain ID, denomination, endpoints, and current version. The tip about transitioning to the new testnet is helpful for users.
Note: The static analysis tool flagged "your" on line 19 as a potential misspelling, but this is a false positive. The usage of "your" is correct in this context.
Tools
LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...should use a new denomination,award
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
22-36
: LGTM: Version history and Prerequisites sections are informative.The version history table and prerequisites section provide necessary information for users. The hardware recommendations and Go installation instructions are clear and helpful.
Suggestion for future improvement: Consider adding more details to the version history table, such as release dates or key features introduced in each version.
38-61
: LGTM: Installation instructions are clear and comprehensive.The installation section provides two well-explained options for users:
- Using a prebuilt binary
- Building from source code
Both options include step-by-step instructions that are clear and easy to follow. The flexibility offered allows users to choose the method that best suits their needs and expertise level.
63-83
: LGTM: Configuration instructions are clear and well-structured.The configuration section provides clear instructions for preparing the genesis file and setting mandatory configuration options. The use of the
PersistentPeers
component is a good practice for maintaining up-to-date peer information.Note: The static analysis tool flagged a bare URL on line 82, but this is within a prop of the React component and doesn't require modification in this context.
Tools
Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
117-126
: LGTM: Start the Node and Next Steps sections are concise and informative.The instructions for starting the node are clear and to the point. The brief mention of next steps for creating a validator provides a good direction for users who want to continue setting up their node as a validator.
docs/developer-docs/docs/build-a-keychain/operate-a-keychain/create-a-keychain.md (3)
15-15
: LGTM: Testnet reference updated correctlyThe change from "Buenavista" to "Chiado" is consistent with the PR objective. The grammar and link are correct.
155-155
: LGTM: Testnet reference updated correctlyThe change from "Buenavista" to "Chiado" is consistent with the PR objective. The grammar and link are correct.
Line range hint
1-157
: Overall: Documentation updated successfullyThe changes in this file consistently update the testnet reference from "Buenavista" to "Chiado". The modifications are in line with the PR objectives, and no grammatical or spelling errors were introduced. The overall structure and content of the document remain intact and coherent.
docs/developer-docs/docs/build-a-keychain/operate-a-keychain/fulfill-requests-from-cli.md
Outdated
Show resolved
Hide resolved
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
Outdated
Show resolved
Hide resolved
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 and nitpick comments (1)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
22-27
: Consider enhancing the version history table.While the current table is clear, it could be more informative. Consider adding columns for release dates and brief descriptions of major changes or features introduced in each version. This would provide users with a better understanding of the project's evolution.
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1 hunks)
Additional context used
Path-based instructions (1)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
LanguageTool
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...should use a new denomination,award
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
Markdownlint
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
82-82: null
Bare URL used(MD034, no-bare-urls)
Additional comments not posted (5)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (5)
1-20
: LGTM: Well-structured overview and frontmatter.The frontmatter and overview section provide clear and essential information about the Chiado testnet. The tip about transitioning to the new testnet is helpful for users.
Tools
LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...should use a new denomination,award
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
28-37
: LGTM: Clear and concise prerequisites.The prerequisites section clearly outlines the hardware requirements and necessary software (Go) version. This information is essential for users preparing to set up a node.
38-61
: LGTM: Comprehensive installation instructions.The installation section provides clear instructions for both prebuilt binary and source code options. The step-by-step guide is easy to follow, ensuring users can successfully install the Warden binary.
63-83
: LGTM: Clear configuration instructions with dynamic content.The configuration section provides clear instructions for preparing the genesis file and setting mandatory configuration options. The use of the
<PersistentPeers>
component is a good practice for maintaining up-to-date information.Regarding the static analysis hint about a bare URL:
The bare URL flagged by the static analysis tool is within a code block and is intentional. No changes are needed.
Tools
Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
117-126
: LGTM: Clear instructions for starting the node and next steps.The instructions for starting the node are clear and concise. The mention of next steps for creating a validator provides a good lead-in to further documentation, helping users understand the progression of tasks.
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.
Looks good to me so far. My only concern is links like rpc, snapshot, faucet will stay same.
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 (2)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
18-20
: Minor grammatical improvement needed in the tip.The tip provides important information about transitioning to the new testnet. However, there's a minor grammatical issue that can be improved.
Consider adding a comma for better readability:
- Chiado is our new and improved testnet. Please make sure to transition all your testing and development processes here. Also note that on Chiado we've changed the denomination to `award`. + Chiado is our new and improved testnet. Please make sure to transition all your testing and development processes here. Also, note that on Chiado we've changed the denomination to `award`.🧰 Tools
🪛 LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Possible missing comma found.
Context: ...pment processes here. Also note that on Chiado we've changed the denomination to `awar...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...ve changed the denomination toaward
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
docs/developer-docs/docs/operate-a-node/create-a-validator.md (1)
7-11
: LGTM! Minor punctuation fix needed.The updates to the introduction and the added tip box clearly communicate the transition from Buenavista to Chiado testnet. This change provides valuable guidance for users.
Consider removing the period at the end of line 10 for consistency with the bullet point style:
-We're transitioning from [Buenavista](buenavista-testnet/join-buenavista) to our new and improved testnet, [Chiado](chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we're going to sunset Buenavista. Please [join Chiado](chiado-testnet/join-chiado) and set up your validator here – then you can safely shut down your validators on Buenavista. +We're transitioning from [Buenavista](buenavista-testnet/join-buenavista) to our new and improved testnet, [Chiado](chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we're going to sunset Buenavista. Please [join Chiado](chiado-testnet/join-chiado) and set up your validator here – then you can safely shut down your validators on Buenavista🧰 Tools
🪛 LanguageTool
[uncategorized] ~10-~10: Loose punctuation mark.
Context: ...hut down your validators on Buenavista. ::: ## Prerequisites The following inst...(UNLIKELY_OPENING_PUNCTUATION)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (2)
- docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1 hunks)
- docs/developer-docs/docs/operate-a-node/create-a-validator.md (4 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"docs/developer-docs/docs/operate-a-node/create-a-validator.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
🪛 LanguageTool
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Possible missing comma found.
Context: ...pment processes here. Also note that on Chiado we've changed the denomination to `awar...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...ve changed the denomination toaward
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
docs/developer-docs/docs/operate-a-node/create-a-validator.md
[uncategorized] ~10-~10: Loose punctuation mark.
Context: ...hut down your validators on Buenavista. ::: ## Prerequisites The following inst...(UNLIKELY_OPENING_PUNCTUATION)
🪛 Markdownlint
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
82-82: null
Bare URL used(MD034, no-bare-urls)
🔇 Additional comments (11)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (8)
22-37
: LGTM: Clear version history and prerequisites.The version history table and prerequisites section are well-structured and provide clear information. The hardware recommendations are specific, which is helpful for users setting up their nodes.
38-61
: LGTM: Comprehensive installation instructions.The installation section provides clear and detailed instructions for both prebuilt binary and source code options. The step-by-step approach and use of code blocks make it easy for users to follow.
63-84
: LGTM: Clear configuration instructions.The configuration section provides clear instructions for preparing the genesis file and setting mandatory configuration options. The use of the custom component for setting persistent peers is a good approach for maintainability.
🧰 Tools
🪛 Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
85-116
: LGTM: Comprehensive state sync setup instructions.The state sync setup section provides detailed and clear instructions. The use of environment variables for configuration is a good practice. The previously reported typo in the curl command has been fixed, addressing the past review comment.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
117-122
: LGTM: Clear instructions for starting the node.The "Start the node" section provides concise instructions and a helpful explanation of what to expect after starting the node. This is valuable information for users setting up their nodes.
124-126
: LGTM: Clear next steps provided.The "Next steps" section provides a clear direction for users who want to create a validator in the testnet. The use of a relative link is good for maintainability.
82-82
: Verify the use of bare URL in custom component.A bare URL is used within the custom
<PersistentPeers>
component. While this is typically discouraged in Markdown, it may be intentional for passing the URL as a prop to the component.Could you confirm if the bare URL usage is intentional and required for the
<PersistentPeers>
component to function correctly? If not, consider enclosing the URL in angle brackets to adhere to Markdown best practices.🧰 Tools
🪛 Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
1-126
: Overall: Well-structured and comprehensive documentation.This documentation provides clear and detailed instructions for joining the Chiado testnet. It covers all necessary steps from installation to configuration and starting the node. The content is well-organized, making it easy for users to follow the process.
A few minor improvements were suggested:
- Adding a comma in the tip section for better readability.
- Verifying the intentional use of a bare URL in a custom component.
The previously reported issue with the curl command has been addressed, demonstrating responsiveness to feedback.
Great job on creating this comprehensive guide for users!
🧰 Tools
🪛 LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Possible missing comma found.
Context: ...pment processes here. Also note that on Chiado we've changed the denomination to `awar...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...ve changed the denomination toaward
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
🪛 Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
docs/developer-docs/docs/operate-a-node/create-a-validator.md (3)
15-15
: LGTM! Prerequisites updated correctly.The prerequisite section has been appropriately updated to reference the Chiado testnet, maintaining consistency with the overall changes in the document.
80-80
: LGTM! Validator creation parameters updated. Please verify the new values.The validator creation parameters have been updated to reflect the Chiado testnet. The changes include:
- Updated amount: from "1000000uward" to "1000000000000000000award"
- New chain-id: "chiado_10010-1"
- Updated fees: from "500uward" to "500000000000000award"
These changes are consistent with the transition to the new testnet. However, the significant increase in amount and fees suggests a change in token denomination or decimal places.
Please run the following script to verify the correctness of the new values:
#!/bin/bash # Description: Verify the correctness of the new validator creation parameters # Test: Check if the chain-id is correct wardend query chain-id # Expected result: chiado_10010-1 # Test: Verify the minimum validator creation amount wardend query staking params | grep -i "min_self_delegation" # Expected result: Should be less than or equal to "1000000000000000000award" # Test: Check the minimum gas price wardend query comet min-gas-price # Expected result: Should be consistent with the new fee structureIf any of these checks fail or produce unexpected results, please review and update the values accordingly.
Also applies to: 100-101
48-48
: LGTM! Faucet URL updated. Please verify the new URL.The faucet URL has been correctly updated to reflect the Chiado testnet. This change is consistent with the overall transition.
Please run the following script to verify that the new faucet URL is accessible and functioning correctly:
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 (1)
docs/developer-docs/docs/operate-a-node/create-a-validator.md (1)
7-11
: LGTM! Minor punctuation fix needed.The updates accurately reflect the transition to Chiado testnet and provide clear guidance for users. However, there's a minor punctuation issue that needs addressing.
Please remove the extra period at the end of line 11:
-We're transitioning from [Buenavista](buenavista-testnet/join-buenavista) to our new and improved testnet, [Chiado](chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we're going to sunset Buenavista. Please [join Chiado](chiado-testnet/join-chiado) and set up your validator here – then you can safely shut down your validators on Buenavista. +We're transitioning from [Buenavista](buenavista-testnet/join-buenavista) to our new and improved testnet, [Chiado](chiado-testnet/join-chiado). For now, both networks are running simultaneously, but we're going to sunset Buenavista. Please [join Chiado](chiado-testnet/join-chiado) and set up your validator here – then you can safely shut down your validators on Buenavista🧰 Tools
🪛 LanguageTool
[uncategorized] ~10-~10: Loose punctuation mark.
Context: ...hut down your validators on Buenavista. ::: ## Prerequisites The following inst...(UNLIKELY_OPENING_PUNCTUATION)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
- docs/developer-docs/docs/operate-a-node/create-a-validator.md (6 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
docs/developer-docs/docs/operate-a-node/create-a-validator.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
🪛 LanguageTool
docs/developer-docs/docs/operate-a-node/create-a-validator.md
[uncategorized] ~10-~10: Loose punctuation mark.
Context: ...hut down your validators on Buenavista. ::: ## Prerequisites The following inst...(UNLIKELY_OPENING_PUNCTUATION)
🔇 Additional comments (4)
docs/developer-docs/docs/operate-a-node/create-a-validator.md (4)
15-15
: LGTM! Prerequisites updated correctly.The prerequisites section has been accurately updated to reference joining the Chiado testnet instead of Buenavista.
66-66
: LGTM! Validator creation parameters updated. Please verify values.The validator creation section has been updated to reflect the Chiado testnet parameters. The changes include:
- Updated command to show validator public key
- New validator creation amount
- Updated chain-id
- Updated transaction fee
Please run the following script to verify the correctness of these values:
#!/bin/bash # Description: Verify Chiado testnet parameters # Test: Check if the 'show-validator' command is correct wardend tendermint show-validator # Test: Verify the chain-id wardend status | jq -r '.NodeInfo.Network' # Test: Check minimum validator creation amount and transaction fee wardend query staking params --output json | jq -r '.bond_denom, .min_commission_rate' # Expected results: # 1. The 'show-validator' command should execute without errors # 2. The chain-id should be "chiado_10010-1" # 3. The bond denomination should be "award" and the minimum commission rate should be consistent with the documentation # If any results are different, please review and update the corresponding values in the documentationAlso applies to: 80-80, 100-101
124-124
:⚠️ Potential issueInconsistency detected in validator check command.
There's an inconsistency between this command and the earlier change in the "Create a validator" section. Earlier, we changed from
comet
totendermint
, but here we're changing fromtendermint
tocomet
.Please verify the correct command and update it consistently throughout the document. If
tendermint
is the correct term, update this line as follows:-wardend query tendermint-validator-set | grep "$(wardend comet show-address)" +wardend query tendermint-validator-set | grep "$(wardend tendermint show-address)"Please run the following script to verify the correct command:
#!/bin/bash # Description: Verify the correct command for showing validator address # Test: Check both commands echo "Testing 'comet' command:" wardend comet show-address echo "Testing 'tendermint' command:" wardend tendermint show-address # Expected result: One of these commands should work without errors # Update the documentation to use the command that works correctly
48-48
: LGTM! Faucet URL updated. Please verify.The faucet URL has been correctly updated to reflect the Chiado testnet.
Please run the following script to verify that the faucet URL is correct and operational:
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 (2)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (2)
38-61
: LGTM with a minor suggestion: Clear installation instructions.The installation section provides two clear options for installing
wardend
. The instructions are easy to follow and use specific version tags for reproducibility.Consider adding a brief explanation or link to documentation for the
just
command used in the source code installation option, as some users might not be familiar with it.
63-84
: LGTM with a suggestion: Clear configuration steps.The configuration section provides clear steps for setting up
wardend
, including preparing the genesis file and setting mandatory configuration options. The use of the custom<PersistentPeers>
component for dynamic configuration is a good approach.Consider adding brief explanations for the
sed
commands used in this section. While these commands are efficient for automation, some users might find them intimidating or difficult to understand at first glance.🧰 Tools
🪛 Markdownlint
82-82: null
Bare URL used(MD034, no-bare-urls)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
- docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (1)
Pattern
**/*.md
: "Assess the documentation for misspellings, grammatical errors, missing documentation and correctness"
🪛 LanguageTool
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...ve changed the denomination toaward
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
🪛 Markdownlint
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md
82-82: null
Bare URL used(MD034, no-bare-urls)
🔇 Additional comments (4)
docs/developer-docs/docs/operate-a-node/chiado-testnet/join-chiado.md (4)
9-20
: LGTM: Clear and informative overview.The overview section effectively provides key information about the Chiado testnet, including the chain ID, denomination, and current version. The tip emphasizing the transition to the new testnet is well-placed and informative.
🧰 Tools
🪛 LanguageTool
[misspelling] ~19-~19: Did you mean “you're” (short for ‘you are’)?
Context: ...net. Please make sure to transition all your testing and development processes here....(YOUR)
[uncategorized] ~19-~19: Loose punctuation mark.
Context: ...ve changed the denomination toaward
. ::: ## Version history | Release | | --...(UNLIKELY_OPENING_PUNCTUATION)
28-36
: LGTM: Clear prerequisites.The prerequisites section provides clear hardware recommendations and specifies the required Go version. This information is crucial for users setting up their environment.
85-116
: LGTM: Comprehensive state sync setup instructions.The state sync setup section provides detailed and helpful instructions for users who want to speed up the initial synchronization process. The use of environment variables and
jq
for parsing JSON responses is a good practice. The previously reported typo in the curl command has been correctly fixed.🧰 Tools
🪛 LanguageTool
[uncategorized] ~88-~88: Loose punctuation mark.
Context: ... This step is recommended but optional. ::: To speed up the initial sync, you ca...(UNLIKELY_OPENING_PUNCTUATION)
117-127
: LGTM: Clear instructions for starting the node and next steps.The instructions for starting the node are clear and concise. The inclusion of next steps with a link to validator creation instructions is helpful for users who want to proceed further in their journey with the Chiado testnet.
Summary by CodeRabbit
New Features
Changes
Bug Fixes
Chores