Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Ruff: Add and autofix B028 #12024

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 27, 2025
Merged

Ruff: Add and autofix B028 #12024

merged 3 commits into from
Mar 27, 2025

Conversation

kiblik
Copy link
Contributor

@kiblik kiblik commented Mar 16, 2025

Add rule no-explicit-stacklevel (B028) and use autofix.

@github-actions github-actions bot added the lint label Mar 16, 2025
Copy link

dryrunsecurity bot commented Mar 16, 2025

DryRun Security Summary

Minor configuration updates in DefectDojo project involving warning mechanism adjustments and linter rule enhancements without introducing security vulnerabilities.

Expand for full summary

Summary: Minor configuration and warning mechanism updates across multiple files in the DefectDojo project, focusing on deprecation warnings and linting rule enhancements.

Security Findings:

  • No direct security vulnerabilities were identified in the reviewed patches
  • The changes primarily involve:
    • Adding stacklevel=2 to warning messages in importer files
    • Updating Ruff linter configuration with an additional rule (B028)

No concrete security risks were found in the provided summaries.

View PR in the DryRun Dashboard.

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@mtesauro
Copy link
Contributor

@kiblik Ironically, this has a linter issue now 🤷

@kiblik
Copy link
Contributor Author

kiblik commented Mar 26, 2025

@kiblik Ironically, this has a linter issue now 🤷

All new PRs will have this problem. #12108 needs to be merged first.

Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@mtesauro mtesauro merged commit 006c0a2 into DefectDojo:dev Mar 27, 2025
77 checks passed
@kiblik kiblik deleted the ruff/B028 branch March 27, 2025 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants