Skip to content

Conversation

@OpenBagTwo
Copy link
Owner

Summary

Addresses #136 by switching the vast majority of logger calls to use %s-formatting instead of f-strings

List of Changes

  • Updates pre-commit
  • Switches logger calls to use %s-formatting
  • Some general linting

Tech Debt and Other Concerns

Validation Performed

  • lint score is now 9.54/10 whereas it's 9.52/10 on the current dev branch
  • I've reviewed the linter report and found no outstanding issues
  • all unit tests pass running on my local

PR Type

  • This PR introduces a breaking change (will
    require a bump in the minor version)
  • The changes in this PR are high urgency and necessitate a hotfix or patch
    release (will require rebasing off of release)
  • This is a release (staging) PR (maintainer use only)

Checklist:

  • I have read the contributor's guide
  • I have run mkdocs serve locally and ensured that all API docs and
    changes I have made to the static pages are rendering correctly, with all links
    working
  • All tech debt concerns have been resolved, documented as issues, or otherwise
    accepted
  • I agree to license my contribution to this project under
    the GNU Public License v3

@OpenBagTwo OpenBagTwo merged commit ea80921 into dev Jul 30, 2025
13 checks passed
@OpenBagTwo OpenBagTwo deleted the better-logging branch July 30, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants