Skip to content

Add blending for use lazy % formatting in logging functions#767

Merged
Pierre-Sassoulas merged 1 commit intomasterfrom
dupplicated
May 19, 2025
Merged

Add blending for use lazy % formatting in logging functions#767
Pierre-Sassoulas merged 1 commit intomasterfrom
dupplicated

Conversation

@sbrunner
Copy link
Copy Markdown
Member

Description

Add blending about Use lazy % formatting in logging functions (PyLint) and
Logging statement uses f-string (Ruff).

Motivation and Context

Better PyLint and Ruff integration

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@sbrunner sbrunner requested a review from Copilot May 19, 2025 12:13
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new blender combination to enforce lazy % formatting in logging functions by integrating specific linting rules for PyLint and Ruff.

  • Added a new configuration item for lazy % formatting in logging functions
  • Configured PyLint to flag f-string interpolation in logging and Ruff to flag a similar case

@sbrunner sbrunner requested a review from Pierre-Sassoulas May 19, 2025 12:16
@Pierre-Sassoulas Pierre-Sassoulas merged commit 389ba51 into master May 19, 2025
7 checks passed
@sbrunner
Copy link
Copy Markdown
Member Author

Thanks :-)

@sbrunner sbrunner deleted the dupplicated branch August 13, 2025 10:08
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.

3 participants