Conversation
fix typo in link
WalkthroughThe update primarily addresses a correction in the documentation by fixing a hyperlink reference. This minor but crucial adjustment ensures that users are directed to the correct resource, enhancing the accuracy and usability of the documentation. Changes
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.
Actionable comments posted: 0
Out of diff range and nitpick comments (9)
developers/blobstream-contracts.md (9)
Line range hint
111-111: Consider adding a comma for clarity: "In other words, analogously, it verifies that a specific block header..."
Line range hint
129-129: Reword for clarity and grammatical correctness: "The library is stateless, and allows passing anIDAOracleinterface as a parameter..."
Line range hint
138-138: Remove the extra space before "This function verifies that the shares..."
Line range hint
144-144: Remove the extra space before "This function verifies that a row/column..."
Line range hint
150-150: Remove the extra space before "This function verifies that a set of..."
Line range hint
156-156: Remove the extra space before "This function computes the Celestia..."
Line range hint
162-162: Remove the extra space before "This function computes the Celestia..."
Line range hint
164-164: Correct the grammatical error: "It is the user's responsibility to verify that the proof is valid and that the shares were successfully committed to using theverifySharesToDataRootTupleRoot()method."- from a shares to row/column root proof. + from shares to a row/column root proof.
Line range hint
166-166: Consider adding a comma for clarity: "...committed to, using theverifySharesToDataRootTupleRoot()method."
Review Details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- developers/blobstream-contracts.md (1 hunks)
Additional Context Used
LanguageTool (9)
developers/blobstream-contracts.md (9)
Near line 111: Possible missing comma found.
Context: ...ndexed by batch nonce). In other words, analogously it verifies that a specific block heade...
Near line 129: Did you mean “passing”? Or maybe you should add a pronoun? In active voice, ‘allow’ + ‘to’ takes an object, usually a pronoun.
Context: .... The library is stateless, and allows to pass anIDAOracle` interface as a parameter...
Near line 138: Loose punctuation mark.
Context: ...rc/lib/verifier/DAVerifier.sol#L80-L124): This function verifies that the sha...
Near line 144: Loose punctuation mark.
Context: ...c/lib/verifier/DAVerifier.sol#L133-L155): This function verifies that a row/c...
Near line 150: Loose punctuation mark.
Context: ...c/lib/verifier/DAVerifier.sol#L164-L194): This function verifies that a set o...
Near line 156: Loose punctuation mark.
Context: ...c/lib/verifier/DAVerifier.sol#L204-L215): This function computes the Celestia ...
Near line 162: Loose punctuation mark.
Context: ...c/lib/verifier/DAVerifier.sol#L224-L229): This function computes the Celestia...
Near line 164: The plural noun “shares” cannot be used with the article “a”. Did you mean “a share” or “shares”?
Context: ...s the Celestia block square size from a shares to row/column root proof. It is the u...
Near line 166: Possible missing comma found.
Context: ... the shares were successfully committed to using the `verifySharesToDataRootTupl...
Additional comments not posted (1)
developers/blobstream-contracts.md (1)
170-170: Corrected hyperlink to point to the appropriate document.
fix typo in link
Overview
Link was broken. Here's a fix.
Summary by CodeRabbit