diff options
author | Sytse Sijbrandij <sytse@gitlab.com> | 2016-12-27 13:22:02 +0100 |
---|---|---|
committer | Sytse Sijbrandij <sytse@gitlab.com> | 2016-12-27 13:22:02 +0100 |
commit | 7daa541668de012b1c232763df0d03e29b5deb7c (patch) | |
tree | 53c54d3759717ea5159ab760aa5f62d899facea8 /CONTRIBUTING.md | |
parent | 07f0486cf8d5952afb76d153888e600f1f26ad87 (diff) | |
download | gitlab-ce-7daa541668de012b1c232763df0d03e29b5deb7c.tar.gz |
Link to k8s contributing doc.
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r-- | CONTRIBUTING.md | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 659871a06a4..b68c4a67826 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -217,8 +217,8 @@ We welcome merge requests with fixes and improvements to GitLab code, tests, and/or documentation. The features we would really like a merge request for are listed with the label [`Accepting Merge Requests` on our issue tracker for CE][accepting-mrs-ce] and [EE][accepting-mrs-ee] but other improvements are also welcome. Please note -that if an issue is marked for the current milestone either before or while you -are working on it, a team member may take over the merge request in order to +that if an issue is marked for the current milestone either before or while you +are working on it, a team member may take over the merge request in order to ensure the work is finished before the release date. If you want to add a new feature that is not labeled it is best to first create @@ -300,6 +300,7 @@ you start with a very simple UI? Can you do part of the refactor? The increased reviewability of small MRs that leads to higher code quality is more important to us than having a minimal commit log. The smaller an MR is the more likely it is it will be merged (quickly). After that you can send more MRs to enhance it. +The ['How to get faster PR reviews' document of Kubernetes](https://github.com/kubernetes/community/blob/master/contributors/devel/faster_reviews.md) also has some great points regarding this. For examples of feedback on merge requests please look at already [closed merge requests][closed-merge-requests]. If you would like quick feedback |