diff options
author | Jiri Podivin <jpodivin@redhat.com> | 2022-08-04 16:32:44 +0200 |
---|---|---|
committer | Jiri Podivin <jpodivin@redhat.com> | 2022-08-04 16:32:44 +0200 |
commit | 56b700afcab95080182687f945c62cc96f650e45 (patch) | |
tree | 5b33320f253e9eacee1147f0dacfa94f0d3ae9e4 | |
parent | a04a48f4f7dc72b1bcc95a5c6a550c7650e35ab3 (diff) | |
download | cliff-56b700afcab95080182687f945c62cc96f650e45.tar.gz |
Removing brackets around tested conditional
While permissible syntactically, using brackets to wrap tested conditional is unnecessary
and potentially confusing. As without inserted whitspace the code may look as a function
call, rather than a statement and expression.
The construct is also considered erroneous by linters.
Closes-Bug: 1983593
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
Change-Id: I0a086a8349e2a72cae024857e148fddc3556c319
-rw-r--r-- | cliff/tests/test_app.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cliff/tests/test_app.py b/cliff/tests/test_app.py index d38861c..bffbb70 100644 --- a/cliff/tests/test_app.py +++ b/cliff/tests/test_app.py @@ -321,7 +321,7 @@ class TestOptionParser(base.TestBase): return parser def take_action(self, parsed_args): - assert(parsed_args.end == '123') + assert parsed_args.end == '123' class MyCommandManager(commandmanager.CommandManager): def load_commands(self, namespace): |