-
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
chore(md): updated readme and contributing guidelines #5101
Conversation
cc @nodejs/website |
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 realize these were moved from previous content, but they're outdated and should be updated.
Going to update this doc asap. Just relaxing today after a long trip. Thank you all for the feedback so far :) BTW, @mikeesto an off-topic thingy, but I sent you a DM on Twitter :) |
cc @richardlau requesting re-review whenever you get some time 🙇 |
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
LGTM 😀 |
This PR introduces changes to the README and the CONTRIBUTION guidelines.
This PR needs a review and approval from @nodejs/website as it changes the Approval Requirements, Commit Guidelines and PR guidelines.