From bb25260f001c741d419f5732da4431a548b0749d Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Thu, 21 Sep 2017 11:48:30 -0400 Subject: flake8: Perform suggested whitespace/formatting changes This only touches newlines, spaces, and (occaisionally) commas. Anything else is left for another commit. --- mesonrewriter.py | 1 - 1 file changed, 1 deletion(-) (limited to 'mesonrewriter.py') diff --git a/mesonrewriter.py b/mesonrewriter.py index 426d878..c8af3a8 100755 --- a/mesonrewriter.py +++ b/mesonrewriter.py @@ -30,4 +30,3 @@ if __name__ == '__main__': print('Warning: This executable is deprecated. Use "meson rewrite" instead.', file=sys.stderr) sys.exit(mesonmain.run(['rewrite'] + sys.argv[1:])) - -- cgit v1.1 From 4dbbb4884c205a1b82ac0deb9c751161a44534fd Mon Sep 17 00:00:00 2001 From: Luke Shumaker Date: Thu, 21 Sep 2017 11:49:00 -0400 Subject: flake8: Clean up complained-about unused imports This also adds a "# noqa: F401" comment on an unused "import lzma", which we are using it in a try/except block that is being used to check if the lzma module is importable; of course it is unused. v2: This turned out to be a little tricky. mesonbuild/modules/__init__.py had the "unused" import: from ..interpreterbase import permittedKwargs, noKwargs However, that meant that the various modules could do things like: from . import noKwargs # "." is "mesonbuild.modules" Which breaks when you remove __init__.py's "unused" import. I could have tagged that import with "# noqa: F401", but instead I chose to have each of the module import directly from "..interpreterbase" instead of ".". --- mesonrewriter.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'mesonrewriter.py') diff --git a/mesonrewriter.py b/mesonrewriter.py index c8af3a8..e6f2637 100755 --- a/mesonrewriter.py +++ b/mesonrewriter.py @@ -23,7 +23,7 @@ # - move targets # - reindent? -from mesonbuild import mesonmain, mlog +from mesonbuild import mesonmain import sys if __name__ == '__main__': -- cgit v1.1