Skip to content

Conversation

@MiConnell
Copy link
Contributor

Description

This PR adds a link to A rollup-centric ethereum roadmap, which is mentioned at the end of the video from the More of a visual learner section

Related Issue

This PR adds a link to A rollup-centric ethereum roadmap, which is mentioned at the end of the video from the `More of a visual learner` section
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @MiConnell! Just noting we do have this article linked on the /en/eth2/shard-chain shard chains page, but agree it provides value on this page as well.

Copy link
Contributor

@minimalsm minimalsm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @MiConnell 💪 Just need a small tweak to enforce the convention we use when referring to article authors.

@minimalsm
Copy link
Contributor

[ ] - Note to self - add this to style guide

@minimalsm
Copy link
Contributor

Thanks for the changes! (I've added hopefully clearer guidance around this in #4773). Merging 🚀 Don't forget to claim claim your POAP for 2021 if you haven't already 🥳!

@minimalsm minimalsm merged commit e8a2889 into ethereum:dev Dec 14, 2021
@MiConnell
Copy link
Contributor Author

awesome, thank you!

@wackerow wackerow mentioned this pull request Dec 16, 2021
@MiConnell MiConnell deleted the patch-2 branch December 17, 2021 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants