Skip to content

Commit 132d83c

Browse files
[v3-1-test] Replace last pylint usage by ruff rule (#57294) (#57307)
(cherry picked from commit 17e9a6d) Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
1 parent b7d69a2 commit 132d83c

2 files changed

Lines changed: 1 addition & 18 deletions

File tree

.pre-commit-config.yaml

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -964,24 +964,6 @@ repos:
964964
- "B101,B301,B324,B403,B404,B603"
965965
- "--severity-level"
966966
- "high" # TODO: remove this line when we fix all the issues
967-
- id: pylint
968-
name: pylint
969-
description: "Pylint is a static code analyser for Python 2 or 3."
970-
entry: pylint
971-
language: python
972-
language_version: python3
973-
types: [python]
974-
additional_dependencies: ['pylint==3.1.0']
975-
require_serial: true
976-
files: ^airflow-core/src/airflow/.*
977-
exclude:
978-
airflow/example_dags/.*
979-
args:
980-
# Use pylint only for the specific check, which are not available into the ruff
981-
- "--disable=all"
982-
# W0133: "Exception statement has no effect"
983-
# see: https://github.com/astral-sh/ruff/issues/10145
984-
- "--enable=W0133"
985967
- id: check-no-fab-migrations
986968
language: pygrep
987969
name: Check no migration is done on FAB related table

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -619,6 +619,7 @@ extend-select = [
619619
"RET506", # Unnecessary {branch} after raise statement
620620
"RET507", # Unnecessary {branch} after continue statement
621621
"RET508", # Unnecessary {branch} after break statement
622+
"PLW0133", # Missing raise statement on exception
622623
]
623624
ignore = [
624625
"D100", # Unwanted; Docstring at the top of every file.

0 commit comments

Comments
 (0)