Skip to content

Cleanup logging and f-strings #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Cleanup logging and f-strings #20

wants to merge 3 commits into from

Conversation

@Lorenyx
Copy link

@Lorenyx Lorenyx commented Aug 10, 2025

Make updates from:

  • Pylint W1309
  • Pylint W1203
  • Consistent "unknown kwargs" logging across logs

Some changes update logging.error to logging.exception to log the exception instead of printing it manually

Lorenyx added 3 commits August 9, 2025 13:53
@Lorenyx
Update unknown kwargs to log as debug
398d3c5
@Lorenyx
Resolve pylint W1309
888af34 
(f-string-without-interpolation)
@Lorenyx
Resolve pylint W1203
82a8446 
(logging-fstring-interpolation)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Lorenyx