aboutsummaryrefslogtreecommitdiff
path: root/CONTRIBUTING
diff options
context:
space:
mode:
authormoneromooo-monero <moneromooo-monero@users.noreply.github.com>2016-10-17 23:17:45 +0100
committermoneromooo-monero <moneromooo-monero@users.noreply.github.com>2016-10-17 23:18:34 +0100
commitbfc1de7aa27a335177cebc11d0575535e8909e8c (patch)
tree5dadf568d25c18965519f408d9e38de48c909959 /CONTRIBUTING
parentMerge pull request #1203 (diff)
downloadmonero-bfc1de7aa27a335177cebc11d0575535e8909e8c.tar.xz
CONTRIBUTING: somes notes about how to contribute
Diffstat (limited to 'CONTRIBUTING')
-rw-r--r--CONTRIBUTING31
1 files changed, 31 insertions, 0 deletions
diff --git a/CONTRIBUTING b/CONTRIBUTING
new file mode 100644
index 000000000..30eb221aa
--- /dev/null
+++ b/CONTRIBUTING
@@ -0,0 +1,31 @@
+Patches are preferably to be sent via a github pull request. If that
+can't be done, patches in "git format-patch" format can be sent
+(eg, posted to fpaste.org with a long enough timeout and a link
+posted to #monero-dev on irc.freenode.net).
+
+Patches should be self contained. A good rule of thumb is to have
+one patch per separate issue, feature, or logical change. Also, no
+other changes, such as random whitespace changes or reindentation.
+Following the code style of the particular chunk of code you're
+modifying is encourgaged. Proper squashing should be done (eg, if
+you're making a buggy patch, then a later patch to fix the bug,
+both patches should be merged).
+
+Commit messages should be sensible. That means a subject line that
+describes the patch, with an optional longer body that gives details,
+documentation, etc.
+
+Comments are encouraged.
+
+If modifying code for which Doxygen headers exist, that header must
+be modified to match.
+
+When submitting a pull request on github, make sure your branch is
+rebased. No merge commits nor stray commits from other people in
+your submitted branch, please. You may be asked to rebase if there
+are conflicts (even trivially resolvable ones).
+
+PGP signing commits is strongly encouraged. That should explain why
+the previous paragraph is here.
+
+Tests would be nice to have if you're adding functionality.