-
Notifications
You must be signed in to change notification settings - Fork 28
chore: add provenance as an input CLI option #654
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
Can you please add this new option to the CLI documentation? |
@@ -125,6 +131,8 @@ def run(self, user_config: dict, sbom_path: str = "", skip_deps: bool = False) - | |||
The path to the SBOM. | |||
skip_deps : bool | |||
Flag to skip dependency resolution. | |||
prov_payload : InToToPayload |
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.
prov_payload : InToToPayload | |
prov_payload : InToToPayload | None |
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.
Fixed in dcb5386
Should the |
Changing the provenance check and the related refactoring can be done in the next PR. |
I have added it in this commit ca2061e |
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.
Looks good to me.
Signed-off-by: Trong Nhan Mai <trong.nhan.mai@oracle.com>
Closes #34 .
This PR includes the following additions:
--provenance-file
is added to received the provenance path from the user.This PR does not implements the validation and extracting information from the provenances. That will be added within feat: use provenance to find commits for supported PURL types. #653.