]> git.ipfire.org Git - thirdparty/knot-resolver.git/commitdiff
lint: pylint and flake8 removed
authorAleš Mrázek <ales.mrazek@nic.cz>
Thu, 10 Oct 2024 12:36:28 +0000 (14:36 +0200)
committerAleš Mrázek <ales.mrazek@nic.cz>
Thu, 17 Oct 2024 12:39:47 +0000 (14:39 +0200)
.gitlab-ci.yml
meson.build
scripts/meson/run-pylint.sh [deleted file]
tests/pytests/pylintrc [deleted file]

index 1415c34ed5efa4833a6bfcda76ec4cc32a62ae2a..2e5d41132a7f46248eb68972b4cf7a285386c9f9 100644 (file)
@@ -245,12 +245,10 @@ trivial_checks: # aggregated to save some processing
     - ci/no_assert_check.sh
     - ci/deckard_commit_check.sh
 
-lint:other:
+lint:luacheck:
   <<: *sanity
   script:
     - meson build_ci_lint &>/dev/null
-    - ninja -C build_ci* pylint
-    - ninja -C build_ci* flake8
     - ninja -C build_ci* luacheck
 
 lint:pedantic:
index 82035cecc4125c73c3ae9fc8a1b8e4785fd5a127..dac67786028df82fa55c1966f38b2951d0970759 100644 (file)
@@ -310,8 +310,6 @@ install_data(
 message('--- lint dependencies ---')
 clangtidy = find_program('clang-tidy', required: false)
 luacheck = find_program('luacheck', required: false)
-flake8 = find_program('flake8', required: false)
-pylint_run = find_program('scripts/meson/run-pylint.sh')
 message('-------------------------')
 
 if clangtidy.found()
@@ -338,22 +336,6 @@ if luacheck.found()
   )
 endif
 
-if flake8.found()
-  run_target(
-    'flake8',
-    command: [
-      flake8,
-      '--max-line-length=100',
-      meson.source_root() / 'tests' / 'pytests',
-    ],
-  )
-endif
-
-run_target(
-  'pylint',
-  command: pylint_run,
-)
-
 
 # Summary message
 # NOTE: ternary operator in format() not supported
diff --git a/scripts/meson/run-pylint.sh b/scripts/meson/run-pylint.sh
deleted file mode 100755 (executable)
index ac23f5c..0000000
+++ /dev/null
@@ -1,12 +0,0 @@
-#!/usr/bin/env bash
-# SPDX-License-Identifier: GPL-3.0-or-later
-set -o errexit -o nounset
-
-cd "$(dirname ${0})/../.."
-
-# Find Python modules and standalone Python scripts
-FILES=$(find ./tests/pytests \
-       -type d -exec test -e '{}/__init__.py' \; -print -prune -o \
-       -name '*.py' -print)
-
-python3 -m pylint -j 0 --rcfile ./tests/pytests/pylintrc ${FILES}
diff --git a/tests/pytests/pylintrc b/tests/pytests/pylintrc
deleted file mode 100644 (file)
index 5e2b50b..0000000
+++ /dev/null
@@ -1,32 +0,0 @@
-# SPDX-License-Identifier: GPL-3.0-or-later
-[MESSAGES CONTROL]
-
-disable=
-    missing-docstring,
-    too-few-public-methods,
-    too-many-arguments,
-    too-many-instance-attributes,
-    fixme,
-    unused-import,  # checked by flake8
-    line-too-long,  # checked by flake8
-    invalid-name,
-    broad-except,
-    global-statement,
-    no-else-return,
-    redefined-outer-name,  # commonly used with pytest fixtures
-    consider-using-with,
-    consider-using-f-string,
-
-
-[SIMILARITIES]
-min-similarity-lines=6
-ignore-comments=yes
-ignore-docstrings=yes
-ignore-imports=no
-
-[DESIGN]
-max-parents=10
-max-locals=20
-
-[TYPECHECK]
-ignored-modules=ssl