From cadf5f1018ad44c1480209f8ec8899c2e2af4f21 Mon Sep 17 00:00:00 2001 From: jichenjc Date: Mon, 26 Jan 2015 07:32:37 +0800 Subject: [PATCH] Remove N331 hacking rules Per discussion in https://review.openstack.org/#/c/145506/ https://review.openstack.org/#/c/135888/ there is no need to check whether warn or warning usage since py3 support both. So according to the suggestion, this patch removes N331 check. Change-Id: I20a62c1d1e953f52a63565b8446f1561d178d5af --- HACKING.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/HACKING.rst b/HACKING.rst index b6db28371..f96b9dfb9 100644 --- a/HACKING.rst +++ b/HACKING.rst @@ -40,7 +40,6 @@ Nova Specific Commandments - [N328] Validate that LOG.info messages use _LI. - [N329] Validate that LOG.exception messages use _LE. - [N330] Validate that LOG.warning messages use _LW. -- [N331] Change LOG.warn on LOG.warning. - [N332] Check that the api_version decorator is the first decorator on a method - [N333] Check for oslo library imports use the non-namespaced packages - [N334] Change assertTrue/False(A in/not in B, message) to the more specific