-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
meta: updated repo structure #5642
Conversation
Signed-off-by: Claudio Wunder <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Signed-off-by: Claudio Wunder <[email protected]>
Signed-off-by: Claudio Wunder <[email protected]>
📦 Next.js Bundle Analysis for nodejs.orgThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Signed-off-by: Claudio Wunder <[email protected]>
Signed-off-by: Claudio Wunder <[email protected]>
Signed-off-by: Claudio Wunder <[email protected]>
cc @nodejs/website as we need a full review here! (As this PR updates the Contributing Guidelines) |
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, just made a couple of small suggestions
Co-authored-by: Michael Esteban <[email protected]> Co-authored-by: Brian Muenzenmeyer <[email protected]> Signed-off-by: Claudio Wunder <[email protected]>
Signed-off-by: Claudio Wunder <[email protected]>
@bmuenzenmeyer I've addressed all comments! |
@ovflowd I unresolved some comments. I am wondering if you intended to stage more changes? there are suggestions made by you that are not committed. |
@bmuenzenmeyer can you double check you're not checking an old version of the diff? Everything is correct over here ;) |
a refresh fixed it 😅 |
I was kinda getting confused lol |
This PR updates relevant sections of the
README
,CONTRIBUTING.md
andCOLLABORATOR_GUIDE.md
with updated guides, references and technologies.cc @nodejs/website