diff options
author | Dylan Baker <dylan@pnwbakers.com> | 2021-08-31 09:55:01 -0700 |
---|---|---|
committer | Eli Schwartz <eschwartz93@gmail.com> | 2021-08-31 16:28:54 -0400 |
commit | 4d7031437c7a81b52c776d4ae1e32741bdb851ca (patch) | |
tree | 7716c4af0d3f43b450a7c94dd42ae5dbef8ebdff /tools | |
parent | 06fdb29daace9ebe55e5df5336f65cba304773d2 (diff) | |
download | meson-4d7031437c7a81b52c776d4ae1e32741bdb851ca.zip meson-4d7031437c7a81b52c776d4ae1e32741bdb851ca.tar.gz meson-4d7031437c7a81b52c776d4ae1e32741bdb851ca.tar.bz2 |
pylint: turn on superflous-parens
We have a lot of these. Some of them are harmless, if unidiomatic, such
as `if (condition)`, others are potentially dangerous `assert(...)`, as
`assert(condtion)` works as expected, but `assert(condition, message)`
will result in an assertion that never triggers, as what you're actually
asserting is `bool(tuple[2])`, which will always be true.
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/build_website.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/build_website.py b/tools/build_website.py index baf81db..50dfc48 100755 --- a/tools/build_website.py +++ b/tools/build_website.py @@ -2,7 +2,7 @@ import os, sys, subprocess, shutil -assert(os.getcwd() == '/home/jpakkane') +assert os.getcwd() == '/home/jpakkane' from glob import glob @@ -37,7 +37,7 @@ def update() -> None: if base == 'CNAME' or base == 'favicon.png': continue subprocess.check_call(['git', 'rm', '-rf', base], cwd=webdir) - assert(os.path.isdir(webdir)) + assert os.path.isdir(webdir) new_entries = glob(os.path.join(htmldir, '*')) for e in new_entries: shutil.move(e, webdir) |