##// END OF EJS Templates
Updated dev docs with information about commit messages.
Fernando Perez -
Show More
@@ -1,52 +1,82
1 1 .. _contributing:
2 2
3 3 ============================
4 4 How to contribute to IPython
5 5 ============================
6 6
7 7 Overview
8 8 ========
9 9
10 10 IPython development is done using Git [Git]_ and Github.com [Github.com]_.
11 11 This makes it easy for people to contribute to the development of IPython.
12 12 There are several ways in which you can join in.
13 13
14 14
15 15 Merging a branch into trunk
16 16 ===========================
17 17
18 18 Core developers, who ultimately merge any approved branch (from themselves,
19 19 another developer, or any third-party contribution) will typically use
20 20 :command:`git merge` to merge the branch into the trunk and push it to the main
21 21 Git repository. There are a number of things to keep in mind when doing this,
22 22 so that the project history is easy to understand in the long run, and that
23 23 generating release notes is as painless and accurate as possible.
24 24
25 25 * When you merge any non-trivial functionality (from one small bug fix to a
26 26 big feature branch), please remember to always edit the appropriate file in
27 27 the :ref:`What's new <whatsnew_index>` section of our documentation.
28 28 Ideally, the author of the branch should provide this content when they
29 29 submit the branch for review. But if they don't it is the responsibility of
30 30 the developer doing the merge to add this information.
31 31
32 32 * When merges are done, the practice of putting a summary commit message in
33 33 the merge is *extremely* useful. It is probably easiest if you simply use
34 34 the same list of changes that were added to the :ref:`What's new
35 35 <whatsnew_index>` section of the documentation.
36 36
37 37 * It's important that we remember to always credit who gave us something if
38 38 it's not the committer. In general, we have been fairly good on this front,
39 39 this is just a reminder to keep things up. As a note, if you are ever
40 40 committing something that is completely (or almost so) a third-party
41 41 contribution, do the commit as::
42 42
43 $ git commit --author="Someone Else"
43 $ git commit --author="Someone Else <who@somewhere.com>"
44 44
45 45 This way it will show that name separately in the log, which makes it even
46 46 easier to spot. Obviously we often rework third party contributions
47 extensively ,but this is still good to keep in mind for cases when we don't
47 extensively, but this is still good to keep in mind for cases when we don't
48 48 touch the code too much.
49 49
50 50
51 51 .. [Git] The Git version control system.
52 52 .. [Github.com] Github.com. http://github.com
53
54
55 Commit messages
56 ===============
57
58 Good commit messages are very important; they provide a verbal account of what
59 happened that is often invaluable for anyone trying to undestand the intent of
60 a commit later on (including the original author!). And git's log command is a
61 very versatile and powerful tool, capable of extracting a lot of information
62 from the commit logs, so it's important that these logs actually have useful
63 information in them.
64
65 In short, a commit message should have the form::
66
67 One line summary.
68 <THIS LINE MUST BE LEFT BLANK>
69 More detailed description of what was done, using multiple lines and even
70 more than one paragraph if needed. For very simple commits this may not be
71 necessary, but non-trivial ones should always have it.
72
73 Closes gh-NNN. # if the commit closes issue NNN on github.
74
75 This format is understood by many git tools that expect a *single line*
76 summary, so please do respect it.
77
78 An excellent reference on commits message is `this blog post`_, please take a
79 moment to read it (it's short but very informative).
80
81 .. _this blog post: http://who-t.blogspot.com/2009/12/on-commit-messages.html
82
General Comments 0
You need to be logged in to leave comments. Login now