lint: Remove custom --no-gitlint option.
zulint already has --skip for this. Signed-off-by: Anders Kaseorg <anders@zulip.com>
This commit is contained in:
parent
c4a78d0832
commit
2e3c65a044
|
@ -15,7 +15,6 @@ EXCLUDED_FILES = [
|
||||||
def run() -> None:
|
def run() -> None:
|
||||||
parser = argparse.ArgumentParser()
|
parser = argparse.ArgumentParser()
|
||||||
add_default_linter_arguments(parser)
|
add_default_linter_arguments(parser)
|
||||||
parser.add_argument('--no-gitlint', action='store_true', help='Disable gitlint')
|
|
||||||
args = parser.parse_args()
|
args = parser.parse_args()
|
||||||
|
|
||||||
linter_config = LinterConfig(args)
|
linter_config = LinterConfig(args)
|
||||||
|
@ -27,10 +26,8 @@ def run() -> None:
|
||||||
description="Static type checker for Python (config: mypy.ini)")
|
description="Static type checker for Python (config: mypy.ini)")
|
||||||
linter_config.external_linter('flake8', ['flake8'], ['py'],
|
linter_config.external_linter('flake8', ['flake8'], ['py'],
|
||||||
description="Standard Python linter (config: .flake8)")
|
description="Standard Python linter (config: .flake8)")
|
||||||
|
linter_config.external_linter('gitlint', ['tools/lint-commits'],
|
||||||
if not args.no_gitlint:
|
description="Git Lint for commit messages")
|
||||||
linter_config.external_linter('gitlint', ['tools/lint-commits'],
|
|
||||||
description="Git Lint for commit messages")
|
|
||||||
|
|
||||||
@linter_config.lint
|
@linter_config.lint
|
||||||
def custom_py() -> int:
|
def custom_py() -> int:
|
||||||
|
|
|
@ -2,5 +2,5 @@
|
||||||
|
|
||||||
set -ev
|
set -ev
|
||||||
|
|
||||||
tools/lint --no-gitlint
|
tools/lint --skip=gitlint
|
||||||
tools/run-mypy
|
tools/run-mypy
|
||||||
|
|
Loading…
Reference in a new issue