Difference between revisions of "Documentation/Nightly/Developers/Tutorials/ContributePatch"
(One intermediate revision by the same user not shown) | |||
Line 9: | Line 9: | ||
<li> | <li> | ||
− | <p>Create an issue in the [http://na-mic.org/Mantis/bug_report_page.php tracker]. For example, see issue [http://na-mic.org/Mantis/view.php?id=1906 #1906]</p> | + | <p>Create an issue in the [http://na-mic.org/Mantis/bug_report_page.php tracker] setting <i>severity</i> to <i>feature</i>. For example, see issue [http://na-mic.org/Mantis/view.php?id=1906 #1906]</p> |
− | + | {{remark|lightgreen|For simple changes (e.g typo, ...), this step is optional.}} | |
− | |||
− | |||
− | |||
</li> | </li> | ||
Line 30: | Line 27: | ||
<li> | <li> | ||
<p>Fix the code or implement your feature, then commit your change(s)</p> | <p>Fix the code or implement your feature, then commit your change(s)</p> | ||
− | + | {{remark|red|Make sure to read the [[Documentation/{{documentation/version}}/Developers/Style_Guide|Slicer Coding and Commit Style Guide]]}} | |
− | |||
− | |||
</li> | </li> | ||
Latest revision as of 14:43, 31 January 2017
Home < Documentation < Nightly < Developers < Tutorials < ContributePatch
For the latest Slicer documentation, visit the read-the-docs. |
The following instructions describes the recommended workflow to contribute patch to Slicer code base.
It is assumed that you followed the New community member checklist, have a clone of https://github.com/Slicer/Slicer/ and have setup the development environment as described here.
-
Create an issue in the tracker setting severity to feature. For example, see issue #1906
- For simple changes (e.g typo, ...), this step is optional.
Make sure your clone has a git remote. After forking Slicer repository, replace
jcfr
with your git login.git remote add jcfr git@github.com:jcfr/Slicer.git
Create a topic named
<issuer_number>-a-descriptive-topic-name
. For example:git checkout -b 1906-uninstall-extensions-on-restart
-
Fix the code or implement your feature, then commit your change(s)
- Make sure to read the Slicer Coding and Commit Style Guide
Publish the topic on your fork
git push jcfr 1906-uninstall-extensions-on-restart
Create a Pull Request and add a note to the issue with a link pointing to your topic. For example, see note 1906#c4578