-
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
remove aws scout2 parser #9894
remove aws scout2 parser #9894
Conversation
@Maffooch, this is a followup |
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Powered by DryRun Security |
I will not fix Ruff Linter here as this is already covered here: #9903 |
@manuel-sommer That just got merged so hopefully it's no longer an issue for PRs 🤞 |
09f956f
to
52b46bd
Compare
Thank you for the hint @mtesauro . Rebasing is done. |
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
52b46bd
to
3cdc783
Compare
Done @cneill, thank you for the hint. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
@manuel-sommer This is ready to merge once the merge conflicts are fixed. |
4ad0d26
to
90d95f7
Compare
90d95f7
to
52e16f7
Compare
Ready to get merged @mtesauro |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
* remove scout2 content * remove aws_scout2 components
As we don't have a clear timeline for 3.X.X, we could remove awsscout2 parser earlier as the notification to merge to Scout2 was first published in January 2023.