Skip to content

Commit

Permalink
Fix new lint warnings for pylint 2.16
Browse files Browse the repository at this point in the history
  • Loading branch information
jgosmann committed Feb 9, 2023
1 parent 6d4ecff commit d9715b0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
8 changes: 4 additions & 4 deletions dmarc_metrics_exporter/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,10 @@ def main(argv: Sequence[str]):
logging.config.dictConfig(
configuration.get(
"logging",
dict(
version=1,
disable_existing_loggers=False,
),
{
"version": 1,
"disable_existing_loggers": False,
},
)
)

Expand Down
1 change: 1 addition & 0 deletions dmarc_metrics_exporter/tests/test_imap_queue.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ async def test_error_handling_when_processing_queue_message(greenmail):

async def handler(_queue_msg: EmailMessage, is_done=is_done):
is_done.set()
# pylint: disable=broad-exception-raised
raise Exception("Error raised on purpose.")

# When
Expand Down
4 changes: 2 additions & 2 deletions pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -526,5 +526,5 @@ min-public-methods=2

# Exceptions that will emit a warning when being caught. Defaults to
# "BaseException, Exception".
overgeneral-exceptions=BaseException,
Exception
overgeneral-exceptions=builtins.BaseException,
builtins.Exception

0 comments on commit d9715b0

Please sign in to comment.