Contributing pull-requests (PRs)

All work is submitted via Pull Requests.
Pull Requests can be submitted as soon as there is code worth discussing. Pull Requests track the branch, so you can continue to work after the PR is submitted. Review and discussion can begin well before the work is complete, and the more discussion the better. The worst case is that the PR is closed.
Pull Requests should generally be made against master
Pull Requests should be tested, if feasible:
- bugfixes should include regression tests
- new behavior should at least get minimal exercise
Use a descriptive prefix for your PR: ENH (enhancement), FIX, TST, DOC, STY, REF (refactor), WIP (Work in progress)
The person who accepts/merges your PR will include an update to the CHANGES file: prefix: description (URL of pull request)
Run make check-before-commit before submitting the PR. This will require you to either install or be in developer mode with: python setup.py install/develop.
In general, do not catch exceptions without good reason.
- catching non-fatal exceptions. Log the exception as a warning.
- adding more information about what may have caused the error. Raise a new exception using raise_from(NewException("message"), oldException) from future. Do not log this, as it creates redundant/confusing logs.

Contributing issues

When opening a new Issue, please take the following steps:

Search GitHub and/or Neurostars for your issue to avoid duplicate reports. Keyword searches for your error messages are most helpful.
If possible, try updating to master and reproducing your issue, because we may have already fixed it.
OS and version
Nipype version
Output of: import nipype; nipype.get_info()
Versions of underlying tools (e.g., ANTS, FSL, SPM, etc.,.)
Any script, or output log, in a gist (gist.github.com)
When applicable, and where possible, pointers to relevant data files.