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 | |
parent | 07f0486cf8d5952afb76d153888e600f1f26ad87 (diff) | |
download | gitlab-ce-7daa541668de012b1c232763df0d03e29b5deb7c.tar.gz |
Link to k8s contributing doc.
-rw-r--r-- | CONTRIBUTING.md | 5 | ||||
-rw-r--r--[l---------] | vendor/gitignore/Clojure.gitignore | 14 |
2 files changed, 16 insertions, 3 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 diff --git a/vendor/gitignore/Clojure.gitignore b/vendor/gitignore/Clojure.gitignore index 7657a270c45..a9fe6fba80d 120000..100644 --- a/vendor/gitignore/Clojure.gitignore +++ b/vendor/gitignore/Clojure.gitignore @@ -1 +1,13 @@ -Leiningen.gitignore
\ No newline at end of file +pom.xml +pom.xml.asc +*.jar +*.class +/lib/ +/classes/ +/target/ +/checkouts/ +.lein-deps-sum +.lein-repl-history +.lein-plugins/ +.lein-failures +.nrepl-port |