Remove unecessary --fix-imports lint flag
This commit is contained in:
+2
-10
@@ -41,13 +41,6 @@ def test(coverage, filter):
|
|||||||
|
|
||||||
|
|
||||||
@click.command()
|
@click.command()
|
||||||
@click.option(
|
|
||||||
"-f",
|
|
||||||
"--fix-imports",
|
|
||||||
default=True,
|
|
||||||
is_flag=True,
|
|
||||||
help="Fix imports using isort, before linting",
|
|
||||||
)
|
|
||||||
@click.option(
|
@click.option(
|
||||||
"-c",
|
"-c",
|
||||||
"--check",
|
"--check",
|
||||||
@@ -55,7 +48,7 @@ def test(coverage, filter):
|
|||||||
is_flag=True,
|
is_flag=True,
|
||||||
help="Don't make any changes to files, just confirm they are formatted correctly",
|
help="Don't make any changes to files, just confirm they are formatted correctly",
|
||||||
)
|
)
|
||||||
def lint(fix_imports, check):
|
def lint(check):
|
||||||
"""Lint and check code style with black, flake8 and isort."""
|
"""Lint and check code style with black, flake8 and isort."""
|
||||||
skip = [
|
skip = [
|
||||||
"requirements",
|
"requirements",
|
||||||
@@ -85,7 +78,6 @@ def lint(fix_imports, check):
|
|||||||
if check:
|
if check:
|
||||||
isort_args.append("--check")
|
isort_args.append("--check")
|
||||||
black_args.append("--check")
|
black_args.append("--check")
|
||||||
if fix_imports:
|
execute_tool("Fixing import order", "isort", *isort_args)
|
||||||
execute_tool("Fixing import order", "isort", *isort_args)
|
|
||||||
execute_tool("Formatting style", "black", *black_args)
|
execute_tool("Formatting style", "black", *black_args)
|
||||||
execute_tool("Checking code style", "flake8")
|
execute_tool("Checking code style", "flake8")
|
||||||
|
|||||||
Reference in New Issue
Block a user