Skip to content

Conversation

@konippi
Copy link
Contributor

@konippi konippi commented Nov 1, 2025

Summary

Changes

Please provide a summary of what's being changed

  • Replacing f-strings with lazy % formatting in all logging statements
  • Adding ruff's G rule (flake8-logging-format) to automatically enforce logging

User experience

Please share what the user experience looks like before and after this change

No changes.

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have reviewed the contributing guidelines
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Is this a breaking change? (Y/N)

  • Yes
  • No

Please add details about how this change was tested.

  • Did integration tests succeed?
  • If the feature is a new use case, is it necessary to add a new integration test case?

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@konippi konippi requested a review from a team as a code owner November 1, 2025 08:41
Copy link
Contributor

@arangatang arangatang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@wzxxing wzxxing merged commit c4666cc into aws:main Nov 3, 2025
3 checks passed
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.

4 participants