Skip to content

Commit

Permalink
style: Update linting for newest pylint / edx-lint
Browse files Browse the repository at this point in the history
  • Loading branch information
bmtcril committed Sep 23, 2024
1 parent ba99fbf commit 909e2d6
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
2 changes: 1 addition & 1 deletion edx_event_bus_redis/internal/consumer.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ def _consume_indefinitely(self):
'consumer_name': self.consumer_name,
}

try: # pylint: disable=too-many-nested-blocks
try:
logger.info(f"Running consumer for {run_context!r}")

# How many errors have we seen in a row? If this climbs too high, exit with error.
Expand Down
8 changes: 5 additions & 3 deletions pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@
# SERIOUSLY.
#
# ------------------------------
# Generated by edx-lint version: 5.3.0
# Generated by edx-lint version: 5.3.6
# ------------------------------
[MASTER]
ignore = migrations
Expand Down Expand Up @@ -259,6 +259,7 @@ enable =
useless-suppression,
disable =
bad-indentation,
broad-exception-raised,
consider-using-f-string,
duplicate-code,
file-ignored,
Expand Down Expand Up @@ -376,6 +377,7 @@ max-parents = 7
max-attributes = 7
min-public-methods = 2
max-public-methods = 20
max-positional-arguments = 10

[IMPORTS]
deprecated-modules = regsub,TERMIOS,Bastion,rexec
Expand All @@ -384,6 +386,6 @@ ext-import-graph =
int-import-graph =

[EXCEPTIONS]
overgeneral-exceptions = Exception
overgeneral-exceptions = builtins.Exception

# 46c7367b47aae914d36c17c49c676bffbc597fcd
# 4cac938bf878bc7ac3b1c8ca5875b8dc6e1088d4
3 changes: 3 additions & 0 deletions pylintrc_tweaks
Original file line number Diff line number Diff line change
Expand Up @@ -9,3 +9,6 @@ disable+=
django-not-configured,
consider-using-with,
bad-option-value,

[DESIGN]
max-positional-arguments = 10

0 comments on commit 909e2d6

Please sign in to comment.