From ec1f00a07c27f1da3adfa2fed51f5bb0169a870d Mon Sep 17 00:00:00 2001 From: Joe Gordon Date: Mon, 24 Jun 2013 15:02:12 -0700 Subject: [PATCH] Add note about checks have community support This project is only trying to enforce the agreed upon style guide, not dictate it. We do not want a new check added if there is any doubt the check from the community. Because this project has very a limited number of reviewers we have to be extra careful about dictating style to the rest of the community. Consensus in hacking reviewers is not community consensus. Change-Id: Id7aaefb8fb98b365f45c8eeb3aa53de02b9cb4e6 --- README.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/README.rst b/README.rst index 7349ebc..e333035 100644 --- a/README.rst +++ b/README.rst @@ -19,6 +19,7 @@ Each check is a pep8 plugin so read Requirements ------------ +- The check must already have community support. We do not want to dictate style, only enforce it. - The canonical source of the OpenStack Style Commandments is `HACKING.rst `_, and hacking just enforces them; so when adding a new check, it must be in HACKING.rst - False negatives are ok, but false positives are not - Cannot be project specific, project specific checks should be `Local Checks`_