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

Skip to content

Ruff: Add B018 rule #12110

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Apr 7, 2025
Merged

Ruff: Add B018 rule #12110

merged 6 commits into from
Apr 7, 2025

Conversation

manuel-sommer
Copy link
Contributor

@manuel-sommer manuel-sommer commented Mar 26, 2025 •
edited
Loading

No description provided.

@dryrunsecurity DryRunSecurity
Copy link

dryrunsecurity bot commented Mar 26, 2025 •
edited
Loading

DryRun Security Summary

DefectDojo project files received routine code maintenance and configuration updates with no identified security vulnerabilities.

Expand for full summary

Summary: Multiple files in the DefectDojo project underwent minor code cleanup and configuration updates across various components, with no significant security vulnerabilities detected.

Security Findings:
• No concrete security vulnerabilities were found in the reviewed patches
• The changes primarily consisted of code cleanup, linting configuration updates, and minor functional improvements
• Each patch underwent careful review and did not introduce any direct security risks

Code Analysis

We ran 7 analyzers against 7 files and 1 analyzer had findings. 6 analyzers had no findings.

Analyzer Findings
Configured Codepaths Analyzer 2 findings

Overall Riskiness

🔴 Risk threshold exceeded.

We've notified @mtesauro.

View PR in the DryRun Dashboard.

@github-actions GitHub Actions
Copy link
Contributor

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

@github-actions GitHub Actions
Copy link
Contributor

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

@Maffooch Maffooch added this to the 2.45.0 milestone Mar 31, 2025
@manuel-sommer manuel-sommer requested a review from Maffooch March 31, 2025 23:26
dogboat
dogboat requested changes Apr 1, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

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

@manuel-sommer manuel-sommer requested a review from dogboat April 1, 2025 21:12
mtesauro
mtesauro approved these changes Apr 7, 2025
View reviewed changes
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

@mtesauro mtesauro merged commit c9d0896 into DefectDojo:dev Apr 7, 2025
76 of 77 checks passed
@manuel-sommer manuel-sommer deleted the ruff_b018 branch April 7, 2025 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mtesauro mtesauro mtesauro approved these changes

@dogboat dogboat dogboat approved these changes

@hblankenship hblankenship hblankenship approved these changes

@Maffooch Maffooch Maffooch approved these changes

Assignees
No one assigned
Labels
integration_tests lint parser ui
Projects
None yet
Milestone
2.45.0
Development

Successfully merging this pull request may close these issues.
5 participants
@manuel-sommer @mtesauro @dogboat @hblankenship @Maffooch