-
Notifications
You must be signed in to change notification settings - Fork 33
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
feature: nodejs docs #154
feature: nodejs docs #154
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.
Hey @luislucena16 In recent readme update at #148 there were updates that conflicts with this PR. Please update the branch prior to submitting this PR.
hey @0xneves the branch is already up to date! i checked it! |
Yoo @luislucena16, your fork might not be up to date with the mainstream. Check the TL;DR, it's on the top, and yours is on the bottom. |
hey @0xneves I think it's ready, I've checked how it is in |
hey @0xneves done! check it |
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.
Approving the changes after a slightly change on the installation part to make it smaller and linear
hey @0xneves all nodejs documentation is already added!
PR: #139
closes #139