Skip to content

Enabled flake8-debugger plugin rule for ruff linter#5056

Open
JayeshHire wants to merge 5 commits into
open-telemetry:mainfrom
JayeshHire:T10-rule
Open

Enabled flake8-debugger plugin rule for ruff linter#5056
JayeshHire wants to merge 5 commits into
open-telemetry:mainfrom
JayeshHire:T10-rule

Conversation

@JayeshHire
Copy link
Copy Markdown
Contributor

@JayeshHire JayeshHire commented Apr 7, 2026

Description

This rule checks for debugger calls and imports in the production code. The debugger calls and imports inside the production code may cause unintended behaviour such as leak of sensitive data and hanging of the program. This rule is to force contributors from not pushing debugger calls and imports in the production code.

contributes to #4227

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A

uv run ruff check
All checks passed!

Does This PR Require a Contrib Repo Change?

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated

@JayeshHire JayeshHire requested a review from a team as a code owner April 7, 2026 16:25
@github-actions
Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 days of this comment.
If you're still working on this, please add a comment or push new commits.

@github-actions github-actions Bot added the Stale label May 13, 2026
@tammy-baylis-swi tammy-baylis-swi moved this to Ready for review in Python PR digest May 13, 2026
@tammy-baylis-swi
Copy link
Copy Markdown
Contributor

Lgtm, please resolve the merge conflicts here too.

@github-actions github-actions Bot removed the Stale label May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

2 participants