From c9c1770a316af6d103530bedbfa2a807d75859a6 Mon Sep 17 00:00:00 2001 From: Markus Zoeller Date: Tue, 24 Nov 2015 17:37:13 +0100 Subject: [PATCH] add hacking check for config options location This adds a hackign check which raises a pep8 error if a config option was found in location where it shouldn't be. Change-Id: I75df3404cc6178aa179e96d75f154e389c3008af --- HACKING.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/HACKING.rst b/HACKING.rst index 5b769c193..881d8de00 100644 --- a/HACKING.rst +++ b/HACKING.rst @@ -52,6 +52,7 @@ Nova Specific Commandments - [N339] Check common raise_feature_not_supported() is used for v2.1 HTTPNotImplemented response. - [N340] Check nova.utils.spawn() is used instead of greenthread.spawn() and eventlet.spawn() - [N341] contextlib.nested is deprecated +- [N342] Config options should be in the central location ``nova/conf/`` Creating Unit Tests -------------------