-
Notifications
You must be signed in to change notification settings - Fork 370
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
feat(output): add a unified output result #1397
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1397 +/- ##
==========================================
+ Coverage 68.96% 69.05% +0.08%
==========================================
Files 185 186 +1
Lines 17934 18237 +303
==========================================
+ Hits 12368 12593 +225
- Misses 4903 4965 +62
- Partials 663 679 +16 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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! Added some comments.
I'm going to merge this PR first to create the package view for the table output. Feel free to leave comments, and I'll address them in a separate PR. |
Replaces the `HTMLResult` with the unified [output `Result`](#1397) for HTML files.
Refactored the HTMLResult into a unified structure that can be reused by other output formats. This new structure presents results hierarchically, grouping vulnerabilities by ecosystem, source, and package. Compare to the original vulnerabilityResult, it contains more information, such as vulnerability count (called/uncalled/fixable/unfixable/severity), package/vulnerability fix information, etc.
Follow-up PRs: