diff options
| author | Eli Schwartz <eschwartz@archlinux.org> | 2023-02-01 15:57:35 -0500 |
|---|---|---|
| committer | Eli Schwartz <eschwartz@archlinux.org> | 2023-02-01 17:01:30 -0500 |
| commit | d6b81307f66d4d20f7d2c9baac74a8449cad640a (patch) | |
| tree | fafbd6801f3c6a0f6d5d1f48e3e8f67651d26f6d /mesonbuild/scripts | |
| parent | 1000246d071be1cc1899080f55cab41646214a40 (diff) | |
| download | meson-d6b81307f66d4d20f7d2c9baac74a8449cad640a.zip meson-d6b81307f66d4d20f7d2c9baac74a8449cad640a.tar.gz meson-d6b81307f66d4d20f7d2c9baac74a8449cad640a.tar.bz2 | |
pylint 2.16: remove pointless parens around equality assignments
Given the construct `foo = (bar == baz)` some people like parentheses
and some do not. They're pointless and don't mean anything, though. I
don't feel this is particularly helpful to code clarity, tbh, and pylint
now notices this and warns about it in our current pylint config.
I think this is reasonable, so let's remove the odd parens.
Diffstat (limited to 'mesonbuild/scripts')
| -rw-r--r-- | mesonbuild/scripts/vcstagger.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mesonbuild/scripts/vcstagger.py b/mesonbuild/scripts/vcstagger.py index 051a970..c484ee1 100644 --- a/mesonbuild/scripts/vcstagger.py +++ b/mesonbuild/scripts/vcstagger.py @@ -27,7 +27,7 @@ def config_vcs_tag(infile: str, outfile: str, fallback: str, source_dir: str, re new_data = f.read().replace(replace_string, new_string) if os.path.exists(outfile): with open(outfile, encoding='utf-8') as f: - needs_update = (f.read() != new_data) + needs_update = f.read() != new_data else: needs_update = True if needs_update: |