diff options
author | Sytse Sijbrandij <sytses@gmail.com> | 2014-11-24 13:38:02 +0100 |
---|---|---|
committer | Sytse Sijbrandij <sytses@gmail.com> | 2014-11-24 13:38:02 +0100 |
commit | 282ca063e51bbd115e269f1d28894443bff24b33 (patch) | |
tree | f986199637314058af2b649896ea284d45ba966d /CONTRIBUTING.md | |
parent | 335320a2e1df302bbf0a16374ca7248c43562ca2 (diff) | |
download | gitlab-ce-282ca063e51bbd115e269f1d28894443bff24b33.tar.gz |
Definition of done added to the docs.
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r-- | CONTRIBUTING.md | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index a403984ed28..cbc52f3eee0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -82,6 +82,24 @@ Please keep the change in a single MR **as small as possible**. If you want to c 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). If you would like quick feedback on your merge request feel free to mention one of the Merge Marshalls of [the core-team](https://about.gitlab.com/core-team/). Please ensure that your merge request meets the contribution acceptance criteria. +## Definition of done + +If you contribute to GitLab please know that changes involve more than just code. +We have the following [definition of done](http://guide.agilealliance.org/guide/definition-of-done.html). +Please ensure you support the feature you contribute through all of these steps. + +1. Description explaning the relevancy (see following item) +1. Working and clean code that is commented where needed +1. Unit and integration tests that pass on the CI server +1. Documented in the /doc directory +1. Changelog entry added +1. Reviewed and any concerns are addressed +1. Merged by the project lead +1. Added to the release blog article +1. Added to [the website](https://gitlab.com/gitlab-com/www-gitlab-com/) if relevant +1. Community questions answered +1. Answers to questions rediated (in docs/wiki/etc.) + ## Merge request description format 1. What does this MR do? |