Project

General

Profile

Actions

Contributing Code » History » Revision 1

Revision 1/5 | Next »
Alex Afanasyev, 11/25/2015 05:05 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

  • All code submissions should go to our code review system

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 3 suites:

 ./build/unit-tests-core
 ./build/unit-tests-daemon
 ./build/unit-tests-rib

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.

Updated by Alex Afanasyev about 9 years ago · 5 revisions