summaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
diff options
context:
space:
mode:
authorSytse Sijbrandij <sytses@gmail.com>2014-12-23 09:53:17 +0100
committerSytse Sijbrandij <sytses@gmail.com>2014-12-23 09:53:17 +0100
commitc5a1b808393d5b3769db0d65214df1645b69f6bf (patch)
tree2711e5bbd8e385a3d61b0cb4dfc0b054816abf2c /CONTRIBUTING.md
parent73885c31d6153c2b4ce5039970d51f0802df12ba (diff)
downloadgitlab-ce-c5a1b808393d5b3769db0d65214df1645b69f6bf.tar.gz
One developer tip.
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r--CONTRIBUTING.md1
1 files changed, 1 insertions, 0 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index 9531b27089b..be3b8bb5e2d 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -118,6 +118,7 @@ Please ensure you support the feature you contribute through all of these steps.
1. Can merge without problems (if not please merge `master`, never rebase commits pushed to the remote server)
1. Does not break any existing functionality
1. Fixes one specific issue or implements one specific feature (do not combine things, send separate merge requests if needed)
+1. Migrations should do only one thing (eg: either create a table, move data to a new table or remove an old table) to aid retrying on failure
1. Keeps the GitLab code base clean and well structured
1. Contains functionality we think other users will benefit from too
1. Doesn't add configuration options since they complicate future changes