Skip to content
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

Ruff: add pyupgrade #9755

Merged
merged 1 commit into from
Apr 6, 2024
Merged

Ruff: add pyupgrade #9755

merged 1 commit into from
Apr 6, 2024

Conversation

kiblik
Copy link
Contributor

@kiblik kiblik commented Mar 16, 2024

Add support for https://pypi.org/project/pyupgrade/ in Ruff

@github-actions github-actions bot added settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR apiv2 unittests integration_tests ui parser labels Mar 16, 2024
@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.

@dryrunsecurity DryRunSecurity
Copy link

dryrunsecurity bot commented Mar 19, 2024 •
edited

Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.

DryRun Security Status Findings
Authn/Authz Analyzer ❕ 43 findings
Configured Codepaths Analyzer ✅ 0 findings
Sensitive Files Analyzer ✅ 0 findings

Note

🟢 Risk threshold not exceeded.

Tip

Get answers to your security questions. Add a comment in this PR starting with @DryRunSecurity. For example...

@dryrunsecurity What are common security issues with web application cookies?

Powered by DryRun Security

@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.

@github-actions GitHub Actions
Copy link
Contributor

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

@kiblik kiblik force-pushed the ruff_up branch 2 times, most recently from 6d138f3 to 30b6f7d Compare March 25, 2024 21:30
@github-actions GitHub Actions
Copy link
Contributor

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

mtesauro
mtesauro approved these changes Mar 26, 2024
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

@kiblik
Copy link
Contributor Author

kiblik commented Mar 26, 2024

This PR depends on #9823

@kiblik kiblik mentioned this pull request Mar 27, 2024
Merged
@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.

@kiblik kiblik marked this pull request as draft March 27, 2024 17:38
@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.

@kiblik kiblik closed this Mar 30, 2024
@kiblik kiblik reopened this Mar 30, 2024
@kiblik
Copy link
Contributor Author

kiblik commented Mar 30, 2024

This PR is ready to merge

@kiblik kiblik marked this pull request as ready for review March 30, 2024 19:08
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 3, 2024

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 3, 2024

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

@kiblik kiblik closed this Apr 3, 2024
@kiblik kiblik reopened this Apr 3, 2024
@mtesauro mtesauro merged commit a69b592 into DefectDojo:dev Apr 6, 2024
238 of 239 checks passed
@kiblik kiblik deleted the ruff_up branch April 6, 2024 07:07
@kiblik kiblik mentioned this pull request Apr 9, 2024
Merged
hblankenship pushed a commit to hblankenship/django-DefectDojo that referenced this pull request Apr 26, 2024
@kiblik @hblankenship
Ruff: add pyupgrade (DefectDojo#9755)
cac4217
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@grendel513 grendel513 grendel513 approved these changes

@cneill cneill cneill approved these changes

@mtesauro mtesauro mtesauro approved these changes

@Maffooch Maffooch Maffooch approved these changes

Assignees
No one assigned
Labels
apiv2 integration_tests parser settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR ui unittests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@kiblik @grendel513 @cneill @mtesauro @Maffooch