-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Ruff: Add PLR5 and fix PLR5501 #11714
Conversation
DryRun Security SummaryThe pull request involves code refactoring across DefectDojo project files, with no critical security vulnerabilities introduced but containing potential security concerns around JIRA user creation, sensitive data handling, logging practices, and configuration dependencies that require careful management. Expand for full summaryThe pull request contains code refactoring across multiple files in the DefectDojo project, primarily simplifying conditional logic and reducing code complexity. Multiple files were modified with minor changes to improve code readability and structure. Security Findings:
No direct critical security vulnerabilities were introduced, but the code contains several areas that require careful configuration and management to prevent potential information exposure or unauthorized access. Code AnalysisWe ran
Overall Riskiness🟡 Please give this pull request extra attention during review. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Add
PLR5*
rules and fix collapsible-else-if (PLR5501)