diff options
| author | Dmitry Tantsur <dtantsur@protonmail.com> | 2020-03-30 11:37:02 +0200 |
|---|---|---|
| committer | Dmitry Tantsur <dtantsur@protonmail.com> | 2020-03-31 13:40:00 +0000 |
| commit | 1faa3397a6ae1c9d82d8f4ba90134148bc022e61 (patch) | |
| tree | 24de6242cc44bf2b8d108cc7342932d41a4a0669 /ironic/hacking | |
| parent | a3d7d73a69f85b396aad30610294d5ea246a0df7 (diff) | |
| download | ironic-1faa3397a6ae1c9d82d8f4ba90134148bc022e61.tar.gz | |
Fix the remaining hacking issues
Fixes W504 and E117, resulting in some indentation changes.
Also fixes code that exceeds the complexity requirement, that is bumped
to 20 (mostly to avoid refactoring the agent heartbeat call, resulting
in conflicts for the deploy steps work).
Change-Id: I8e49f2c039b0ddfca9138f8e148708b7e8b5df7e
Diffstat (limited to 'ironic/hacking')
| -rw-r--r-- | ironic/hacking/checks.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/ironic/hacking/checks.py b/ironic/hacking/checks.py index 1d59efb37..6c5b49776 100644 --- a/ironic/hacking/checks.py +++ b/ironic/hacking/checks.py @@ -46,9 +46,9 @@ def check_explicit_underscore_import(logical_line, filename): # checking needed once it is found. if filename in UNDERSCORE_IMPORT_FILES: pass - elif (underscore_import_check.match(logical_line) or - custom_underscore_check.match(logical_line)): + elif (underscore_import_check.match(logical_line) + or custom_underscore_check.match(logical_line)): UNDERSCORE_IMPORT_FILES.append(filename) - elif (translated_log.match(logical_line) or - string_translation.match(logical_line)): + elif (translated_log.match(logical_line) + or string_translation.match(logical_line)): yield(0, "N323: Found use of _() without explicit import of _!") |