-
-
Notifications
You must be signed in to change notification settings - Fork 261
Add fireeye vulnerabilities #487 #795
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
Conversation
4270fd9 to
697d742
Compare
|
@ziadhany please run this importer and add logs for that here, and also run the improve command over this imported data and provide the logs for that |
|
@ziadhany please add logs |
|
importer logs : improver logs : |
|
Please see my review comment, otherwise this looks good to me. Please rebase your branch |
b2528e7 to
371e39f
Compare
TG1999
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.
LGTM!
pombredanne
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! just a few nits for your consideration.
edd8f11 to
db26bb8
Compare
pombredanne
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.
LGTM... I just added a notice for good measure.
Fix docstring style Add the License, add a Git Importer to fireeye Remove print logs Add docstring , add test for parse_advisory_data. Add Fireeye importer Reference: aboutcode-org#487 Signed-off-by: ziadhany <ziadhany2016@gmail.com>
Signed-off-by: ziadhany <ziadhany2016@gmail.com>
Signed-off-by: ziadhany <ziadhany2016@gmail.com>
See aboutcode-org#487 Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
See #487