Skip to content

[GCI 111] For logging format, prefer using %s with kwargs instead of builtin formatter \"\".format() or f\"\"" #122

Merged
dedece35 merged 3 commits intomainfrom
PR_84_DDC
Feb 7, 2026
Merged

[GCI 111] For logging format, prefer using %s with kwargs instead of builtin formatter \"\".format() or f\"\"" #122
dedece35 merged 3 commits intomainfrom
PR_84_DDC

Conversation

@dedece35
Copy link
Copy Markdown
Member

@dedece35 dedece35 commented Feb 7, 2026 •
edited
Loading

The current PR replaces the original one : #84

Why ? because no activity by original author.

This current PR upgrade the original PR with a lot of modification to cover a lot of other use cases.

@dedece35 dedece35 self-assigned this Feb 7, 2026
github-code-quality[bot]
github-code-quality bot found potential problems Feb 7, 2026
View reviewed changes
...ngo-python-plugin-test-project/src/detectBadLoggingFormatInterpolationLoggingNonCompliant.py
@@ -0,0 +1,39 @@
import logging
...edengo-python-plugin-test-project/src/detectBadLoggingFormatInterpolationLoggingCompliant.py
@@ -0,0 +1,44 @@
import logging
src/test/resources/checks/detectBadLoggingFormatInterpolation/logging/loggingCompliant.py
@@ -0,0 +1,35 @@
import logging
src/test/resources/checks/detectBadLoggingFormatInterpolation/logging/loggingNonCompliant.py
@@ -0,0 +1,42 @@
import logging
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 7, 2026

Quality Gate Passed Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
85.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@dedece35 dedece35 merged commit 50c3473 into main Feb 7, 2026
10 checks passed
@dedece35 dedece35 deleted the PR_84_DDC branch February 7, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@dedece35 dedece35

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dedece35 @fabienhusseperso