-
Notifications
You must be signed in to change notification settings - Fork 187
Add plumbing to collect packages from SPDX 2.2 files #1117
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 reverts commit 4fc82c5.
|
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
|
src/Microsoft.Sbom.Extensions/IMergeableContentProviderInternal.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.Sbom.Parsers.Spdx30SbomParser/MergeableContentProvider.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.Sbom.Parsers.Spdx22SbomParser/MergeableContentProvider.cs
Outdated
Show resolved
Hide resolved
test/Microsoft.Sbom.Parsers.Spdx22SbomParser.Tests/MergeableContentProviderTests.cs
Outdated
Show resolved
Hide resolved
|
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 is all about adding the
IMergeableContentProvider interface, but includes some housekeeping. Specifically, it restructures the./samples` folder to include both SPDX 2.2 and SPDX 3.0 files. These files were built against the win64 4.0.3 release, using the win 64 tool from the same release. These samples are available for testsIt also adds a functional SPDX 2.2 implementation of
IMergeableContentProvider(enabled via dependency injection) and a placeholder SPDX 3.0 implementation ofIMergeableContentProvider(not yet in dependency injection).The
SbomConsolidationWorkflow.SourceSbomsTempproperty is a test stub that will be removed in the near future.This code does not yet include plumbing to pass the SBOMs discovered during validation, or plumbing to capture the
MergeableContentin the generation workflow.