]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Effective Neutron: add link to low-hanging-fruit bugs
authorarmando-migliaccio <armamig@gmail.com>
Thu, 1 Oct 2015 19:52:46 +0000 (12:52 -0700)
committerarmando-migliaccio <armamig@gmail.com>
Thu, 1 Oct 2015 19:52:46 +0000 (12:52 -0700)
To draw attention to newbies where it matters

Change-Id: Iabab0b4f1e6e901f122c793a64d74fc4412c7e39

doc/source/devref/effective_neutron.rst

index bb88178879b3c659ce71bed71893b9bed90d36b0..5dfafe0792fafb6dfd94265bd99247affa45abce 100644 (file)
@@ -130,6 +130,14 @@ Document common nits and pedantic comments to watch out for.
   validation (pep8 and/or tests) before submitting your patch. If you keep forgetting
   consider installing a git `hook <https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks>`_
   so that Git will do it for you.
+* Sometimes, new contributors want to dip their toes with trivial patches, but we
+  at OpenStack *love* bike shedding and their patches may sometime stall. In
+  some extreme cases, the more trivial the patch, the higher the chances it fails
+  to merge. To ensure we as a team provide/have a frustration-free experience
+  new contributors should be redirected to fixing `low-hanging-fruit bugs <https://bugs.launchpad.net/neutron/+bugs?field.tag=low-hanging-fruit>`_
+  that have a tangible positive impact to the codebase. Spelling mistakes, and
+  docstring are fine, but there is a lot more that is relatively easy to fix
+  and has a direct impact to Neutron users.
 
 Reviewer comments
 ~~~~~~~~~~~~~~~~~