Commit 51ed906d authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'we-hate-options' into 'master'

We Hate Options
parents faafde2d 41adbc0b
...@@ -84,7 +84,7 @@ We will accept a merge requests if it: ...@@ -84,7 +84,7 @@ We will accept a merge requests if it:
* Fixes one specific issue or implements one specific feature (do not combine things, send separate merge requests if needed) * Fixes one specific issue or implements one specific feature (do not combine things, send separate merge requests if needed)
* Keeps the GitLab code base clean and well structured * Keeps the GitLab code base clean and well structured
* Contains functionality we think other users will benefit from too * Contains functionality we think other users will benefit from too
* Doesn't add avoidable configuration options since these complicate future changes * Doesn't add configuration options since these complicate future changes
* Contains a single commit (please use `git rebase -i` to squash commits) * Contains a single commit (please use `git rebase -i` to squash commits)
For examples of feedback on merge requests please look at already [closed merge requests](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests?assignee_id=&label_name=&milestone_id=&scope=&sort=&state=closed). For examples of feedback on merge requests please look at already [closed merge requests](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests?assignee_id=&label_name=&milestone_id=&scope=&sort=&state=closed).
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment