Skip to content

Commit

Permalink
ci: Fix super-linter errors (#795)
Browse files Browse the repository at this point in the history
* Fix linter issues

* Update pylint config file name

* Update release.yml

* Synchronise configuration for linter across lint and release workflows

---------

Co-authored-by: Ankur Banerjee <[email protected]>
  • Loading branch information
filipdjokic and ankurdotb committed Oct 11, 2024
1 parent 930fee5 commit d248619
Show file tree
Hide file tree
Showing 4 changed files with 131 additions and 88 deletions.
46 changes: 45 additions & 1 deletion .github/linters/.pylintrc
Original file line number Diff line number Diff line change
@@ -1 +1,45 @@
disable=C0301,C0114,C0302,C0116
[MESSAGES CONTROL]

# Only show warnings with the listed confidence levels. Leave empty to show
# all. Valid levels: HIGH, CONTROL_FLOW, INFERENCE, INFERENCE_FAILURE,
# UNDEFINED.
confidence=

# Disable the message, report, category or checker with the given id(s). You
# can either give multiple identifiers separated by comma (,) or put this
# option multiple times (only on the command line, not in the configuration
# file where it should appear only once). You can also use "--disable=all" to
# disable everything first and then re-enable specific checks. For example, if
# you want to run only the similarities checker, you can use "--disable=all
# --enable=similarities". If you want to run only the classes checker, but have
# no Warning level messages displayed, use "--disable=all --enable=classes
# --disable=W".
disable=line-too-long,
too-many-lines,
missing-module-docstring,
missing-function-docstring,
logging-fstring-interpolation,
consider-using-f-string,
too-many-branches,
unspecified-encoding,
missing-class-docstring,
broad-exception-caught,
redefined-outer-name,
consider-using-with,
inconsistent-return-statements,
useless-else-on-loop,
no-else-return,
too-many-return-statements,
subprocess-run-check,
too-many-statements,
too-many-locals,
too-many-public-methods,
too-many-instance-attributes,
invalid-name
# Enable the message, report, category or checker with the given id(s). You can
# either give multiple identifier separated by comma (,) or put this option
# multiple time (only on the command line, not in the configuration file where
# it should appear only once). See also the "--disable" option for examples.
enable=


1 change: 1 addition & 0 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ jobs:
LOG_LEVEL: WARN
VALIDATE_ALL_CODEBASE: false
MULTI_STATUS: true
PYTHON_PYLINT_CONFIG_FILE: .pylintrc

VALIDATE_BASH: true
VALIDATE_DOCKERFILE_HADOLINT: true
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,14 @@ jobs:
LOG_LEVEL: WARN
VALIDATE_ALL_CODEBASE: true
MULTI_STATUS: true
PYTHON_PYLINT_CONFIG_FILE: .pylintrc

VALIDATE_BASH: true
VALIDATE_DOCKERFILE_HADOLINT: true
VALIDATE_ENV: true
VALIDATE_GITHUB_ACTIONS: true
VALIDATE_JSON: true
VALIDATE_MARKDOWN: true
VALIDATE_OPENAPI: true
VALIDATE_PYTHON_PYLINT: true
VALIDATE_XML: true
Expand Down
Loading

0 comments on commit d248619

Please sign in to comment.