]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Merge "Enforce log hints"
authorJenkins <jenkins@review.openstack.org>
Sun, 14 Dec 2014 02:58:34 +0000 (02:58 +0000)
committerGerrit Code Review <review@openstack.org>
Sun, 14 Dec 2014 02:58:34 +0000 (02:58 +0000)
1  2 
neutron/hacking/checks.py
neutron/service.py

index 21c138730fb1d513e1f9fb94357a283f65d31325,76e49ad499f151175306566991ad0457efe7c759..e2855e9f429d0b389b5b363c825a176662274178
@@@ -16,20 -16,21 +16,18 @@@ import r
  
  import pep8
  
 -"""
 -Guidelines for writing new hacking checks
 -
 - - Use only for Neutron specific tests. OpenStack general tests
 -   should be submitted to the common 'hacking' module.
 - - Pick numbers in the range N3xx. Find the current test with
 -   the highest allocated number and then pick the next value.
 - - Keep the test method code in the source file ordered based
 -   on the N3xx value.
 - - List the new rule in the top level HACKING.rst file
 - - Add test cases for each new rule to
 -   neutron/tests/unit/test_hacking.py
 -
 -"""
 +# Guidelines for writing new hacking checks
 +#
 +#  - Use only for Neutron specific tests. OpenStack general tests
 +#    should be submitted to the common 'hacking' module.
 +#  - Pick numbers in the range N3xx. Find the current test with
 +#    the highest allocated number and then pick the next value.
 +#  - Keep the test method code in the source file ordered based
 +#    on the N3xx value.
 +#  - List the new rule in the top level HACKING.rst file
 +#  - Add test cases for each new rule to
 +#    neutron/tests/unit/test_hacking.py
  
- log_translation = re.compile(
-     r"(.)*LOG\.(audit|error|info|warn|warning|critical|exception)\(\s*('|\")")
  author_tag_re = (re.compile("^\s*#\s*@?(a|A)uthor"),
                   re.compile("^\.\.\s+moduleauthor::"))
  _all_hints = set(['_', '_LI', '_LE', '_LW', '_LC'])
Simple merge