-
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
ITRocket endpoints for mocha #1597
Conversation
Warning Review failedThe pull request is closed. WalkthroughThe changes introduce new endpoints for the Celestia testnet, enhancing the network's accessibility and functionality. These include endpoints for consensus, RPC, API, gRPC, and an explorer URL. This update aims to streamline interactions with the Celestia testnet by providing dedicated, easily accessible endpoints for various network services. 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 Configration 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
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- nodes/mocha-testnet.md (5 hunks)
Additional context used
LanguageTool
nodes/mocha-testnet.md
[uncategorized] ~14-~14: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...rabica devnet](./arabica-devnet.md) for that as it is designed for development purpo...
[style] ~75-~75: Consider a shorter alternative to avoid wordiness. (IN_ORDER_TO_PREMIUM)
Context: ...nsensus node](./full-consensus-node.md) in order to download full blocks from it. ::: - `p...
Markdownlint
nodes/mocha-testnet.md
127-127: null
Spaces inside link text
Additional comments not posted (5)
nodes/mocha-testnet.md (5)
84-84
: Endpoint addition confirmed.The addition of
celestia-testnet-consensus.itrocket.net
to the consensus endpoints is consistent with the PR's objective to enhance Celestia testnet connectivity.
107-107
: Endpoint addition confirmed.The addition of
celestia-testnet-rpc.itrocket.net
to the RPC endpoints aligns with the intended improvements to the network's accessibility.
131-131
: Endpoint addition confirmed.The addition of
https://celestia-testnet-api.itrocket.net
to the API endpoints is appropriate and enhances the REST API accessibility for the Celestia testnet.
158-158
: Endpoint addition confirmed.The addition of
celestia-testnet-grpc.itrocket.net:443
to the gRPC endpoints is correctly implemented, improving the gRPC framework accessibility for the testnet.
216-216
: Explorer endpoint addition confirmed.The addition of
https://testnet.itrocket.net/celestia
to the list of explorers is correctly implemented, providing users with an additional resource to explore the Mocha testnet.
@@ -81,6 +81,7 @@ full blocks from it. | |||
- `full.consensus.mocha-4.celestia-mocha.com` | |||
- `consensus-full-mocha-4.celestia-mocha.com` | |||
- `rpc-mocha.pops.one` | |||
- `celestia-testnet-consensus.itrocket.net` |
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.
what ports are open?
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.
RPC: 26657
gRPC: 9090
nodes/mocha-testnet.md
Outdated
@@ -103,6 +104,7 @@ Celestia network. The default port is 26657. | |||
- `celestia-t-rpc.noders.services/` | |||
- `rpc-1.testnet.celestia.nodes.guru` | |||
- `rpc-2.testnet.celestia.nodes.guru` | |||
- `celestia-testnet-rpc.itrocket.net` |
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.
what ports are open?
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.
443
Congrats, your important contribution to this open-source project has earned you a GitPOAP! GitPOAP: 2024 Celestia Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
Overview
Summary by CodeRabbit