forked from nodejs/node-v0.x-archive
-
Notifications
You must be signed in to change notification settings - Fork 24
Contributing
isaacs edited this page Mar 6, 2011
·
17 revisions
The only “official” node repository is https://github.com/joyent/node. For code, docs, tests, or anything else, joyent/node is the repository that reflects the official state of the Node.js project. That’s where all patches have to end up if they are going to be in the release, website, or API docs. There are a variety of ways to do this.
- Email patches to [email protected] and post them to the node.js development mailing list. Please add “[PATCH]” in the subject
- Send a pull request via github to joyent/node
- Post an issue to the mailing list describing the problem and paste a url to the patch gist
- Discuss large changes on mailing list before coding
- Javascript code style should follow Google’s JavaScript style guide and be run through the linter. `make jslint` to validate.
- C++ code should follow Google’s C++ style guide and be run through cpplint with `make cpplint`.
- Agree to the contributor agreement
- the first line should be maximum 50 columns
- the second line should blank
- any additional lines should not exceed 80 columns.
- should have the author field properly filled out with your full
name and email address.
To generate patch files:
- clone the git repository
- make your change. Remember to update tests and docs as well as code!
- commit your change
- Use the
git format-patch
command to generate patch files.