Contributing Code » History » Revision 3
Revision 2 (Alex Afanasyev, 11/25/2015 05:18 PM) → Revision 3/5 (Davide Pesavento, 03/03/2017 05:46 PM)
Contributing Code
=================
NDN codebase is open-source and we welcome and encourage contributions from everybody.
In order to ensure design and code quality, we have established a process that includes code review and continuous integration checks on a number of supported platforms.
The following summarizes the process how to contribute to NDN projects (NFD, ndn-cxx, ndn-tools, NFD-android, NLSR, ndnSIM, ChronoSync, ChronoChat, ndn-group-encrypt):
- To inform NDN team about problems or to suggest new features, create issue in the corresponding project on [NDN redmine](http://redmine.named-data.net)
- All code submissions should go to [our code review system](http://gerrit.named-data.net)
### Code Review
The patches submitted for the code review are automatically checked for:
- successful compilation
- successful run of unit tests
- no errors produced while creating API (doxygen) documentation
We will also accept issues and pull requests on GitHub.
However, this may incur additional delay in merging the submitted code, as one of NDN developers will need to submit code to code review process on your behalf.
### Recommended Checks Before Patch Submission
In order to facilitate code review process, please perform the following tests before submitting patches for code review:
#### Check that code and all unit tests compile. For most of our project, the following is the recommended way to configure and build project
./waf configure --debug --with-tests
./waf
#### Check that unit tests do not fail. Refer to project's documentation on how to run unit tests.
NFD has 4 test 3 suites:
./build/unit-tests-core
./build/unit-tests-daemon
./build/unit-tests-rib
./build/unit-tests-tools
ndn-cxx and most other projects have one unit test suite:
./build/unit-tests
#### Check that there are no errors in API (doxygen) documentation
To check for API documentation, doxygen needs to be installed. To build documentation, run
./waf doxygen
All warnings produced by this command must be fixed in order for the automatic continuous integration checks to succeed.
**Note** that `./waf doxygen` will not rebuild documentation if there were no changes in source files. To rebuild documentation without changing source files, run `./waf clean` before running `./waf doxygen`