-
-
Notifications
You must be signed in to change notification settings - Fork 552
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
Pluggable output formats #381
Comments
So for plugins basics there are a few contenders:
All have some pros and cons. In all cases, combining these with setuptools entry points (even if built in) is the way to go IMHO. |
Creating other formats as a post-processing step from JSON also has the advantage of easily getting scan results in different formats from the same scan. Currently, if you would need both JSON and SPDX, you would need to run the scan twice. |
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* thanks to @sschuberth for the review Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
#381 Move scan formatters to a new "formattedcode" module
Note: A plugin architecture is what is needed here. Some refinements on the formats is welcomed but is not the purpose. The purpose is to refactor the existing code for plugins. |
and each plugin for format should ideally be also runnable as its own CLI, with input = scan json (e.g. ABC Data format), output = other format See for ABCD: |
So format plugins have landed in |
This has been completed and merged in develop. @sschuberth you wrote:
Just ping as if we support #789 in the future, then we would need to have one option for each format such that we can specify which file each output should be written to. |
Or, preferably, extend
|
Following discussions with @sschuberth in #375 we should have a simplified and better to handle output formats. The outline would be this:
The text was updated successfully, but these errors were encountered: