-
-
Notifications
You must be signed in to change notification settings - Fork 273
Add rockylinux advisories #1535
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
base: main
Are you sure you want to change the base?
Add rockylinux advisories #1535
Conversation
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
|
@ziadhany please review this pr, I have added the Rockylinux importer and tests for it. |
ziadhany
left a comment
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.
Thanks @ambuj-1211, please see few nits below for your consideration
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
- Modified the severity score collection. - Refined docstrings Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
3666fb1 to
20fd784
Compare
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
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.
@ambuj-1211 Thank you! Please use the new pipeline and resolve the severity and reference issues, run the importer and print the logs.
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
|
@ziadhany I have made the changes and created the rockylinux pipeline code, here are the logs for it. |
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
|
@ziadhany please check this one also I have already made the suggested changes, please tell me if there are any other changes needed. |
|
@ziadhany please review this one I have done most of the things please tell me what changes should I make. |
fixes: #753