-
Notifications
You must be signed in to change notification settings - Fork 187
Pass set of validated SBOMs to consolidation #1119
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
|
This PR changes files in the API project. Does it change any of the API interfaces in any way? Please note that this includes the following types of changes:
Because any of these changes can potentially break a downstream consumer with customized interface implementations, these changes need to be treated as breaking changes. Please do one of the following: Option 1 - Publish this as a breaking change
Option 2 - Refactor the changes to be non-breaking
|
|
/azp run |
|
This PR changes files in the API project. Does it change any of the API interfaces in any way? Please note that this includes the following types of changes:
Because any of these changes can potentially break a downstream consumer with customized interface implementations, these changes need to be treated as breaking changes. Please do one of the following: Option 1 - Publish this as a breaking change
Option 2 - Refactor the changes to be non-breaking
|
|
/azp run |
#1117 included a temporary property that was just for testing purposes. This PR is about removing that temporary property.
SbomConsolidationWorkflow.SourceSbomsTempTuple, but it felt cleaner to replace theTuplewith a new class--this is theConsolidationSourceclasssbomsToValidatetoconsolidationSources, since the same data will flow to both validation and consolidationsPDXFormatDetector. Change them to begin withspdxFormatDetector