-
Notifications
You must be signed in to change notification settings - Fork 386
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
chore: change instances of "qgb" to "Blobstream", update workflow #1115
Conversation
|
By running `yarn import-md`
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.
First pass looks good 👍 👍
Left some cosmetic comments to be addressed. Then, I will make a second pass on the overview
Co-authored-by: CHAMI Rachid <[email protected]>
…into jcs/blobstream
docs/nodes/blobstream-intro.md
Outdated
Ethereum's L2 contract updates its state, it checks with Blobstream to confirm | ||
that data's presence on Celestia. In contrast, a DAC would rely on attestations |
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.
it checks with Blobstream to confirm that data's presence on Celestia
How this? I am not aware of this happening or I don't get this part
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.
reworked in ba3b632
docs/nodes/blobstream-intro.md
Outdated
computation-centric. The scalability costs of a DAC would largely hinge on its | ||
specific design, but it may not achieve the same balance as Blobstream. |
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.
Not sure about this too
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.
reworked in ba3b632
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Co-authored-by: CHAMI Rachid <[email protected]>
Overview
This PR resolves #1113
qgb
to new name, once renamedblobstream-xxx.md
Checklist