-
Notifications
You must be signed in to change notification settings - Fork 13
Minimal EthereumJS Website Update #11
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
Conversation
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 great and most of my comments are nits, (except the bit about a missing reference to the evm package and LES protocol in devp2p).
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.
I think this all looks great, except that we do not mention these packages:
- Common
- (Ethash)
- Evm
- Genesis
I think these three packages should be added.
Already wrote this in a reply to Andrew, I rather would not want this to just make this a complete package list but rather - team-subjectively - focus on the most important stuff we are doing and put this a bit more in the spotlight (I explicitly wanted to get away from this old version a bit, which was just a long and undifferentiated package list with geth.js (lol) side by side with our VM). So for all mentioned libraries I would have some tendency not to take:
This library is rather not so exciting on its own, so not that much of a "teaser" library
What's that? 😂
As answered in the response to Andrews question.
Not an interesting package on its own. |
Also, maybe to mention, this is somewhat in contrast to our monorepo README entrypoint, where just every package is listed in this long table: https://github.com/ethereumjs/ethereumjs-monorepo Didn't want to just reproduce this. |
Ready for re-review. 🙂 |
Ok, ready for re-review. 🙂 |
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.
LGTM!
This updates the https://ethereumjs.github.io/ EthereumJS website on a basic level by adopting to our current project structure, rewriting some intro texts and generally weighting things more according our current priorities.
This is how the site now renders:
Open for review. 🙂