Skip to content

feat: Add license text as evidence#1243

Merged
jkowalleck merged 19 commits intoCycloneDX:mainfrom
cuhland:add-license-text
Dec 17, 2024
Merged

feat: Add license text as evidence#1243
jkowalleck merged 19 commits intoCycloneDX:mainfrom
cuhland:add-license-text

Conversation

@cuhland
Copy link
Contributor

@cuhland cuhland commented Dec 10, 2024

fixes #256

This PR is based on #427, and uses the same implementation as it was implemented in yarn (See CycloneDX/cyclonedx-node-yarn#193)
closes #427

Matthias Schiebel and others added 9 commits December 4, 2024 17:18
Signed-off-by: Matthias Schiebel <msl@compart.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Matthias Schiebel <msl@compart.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <christoph.uhland@bdr.de>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
address design principles and architecture of this project.
Still wip

Signed-off-by: Christoph Uhland <christoph.uhland@bdr.de>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
As suggested in CycloneDX#427 (comment)

Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
@cuhland cuhland requested a review from a team as a code owner December 10, 2024 15:43
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Copy link
Member

@jkowalleck jkowalleck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for all of your efforts, @cuhland .

some changes/refactors are needed, please see my remarks below.

Also, please add the appropriate test-beds. I could expect this can be trickt. let me know if you need help with this.

@jkowalleck jkowalleck added this to the v2.0.0 milestone Dec 10, 2024
@jkowalleck jkowalleck added the enhancement New feature or request label Dec 10, 2024
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
@jkowalleck
Copy link
Member

some code styles are not applied as expected. see https://github.com/CycloneDX/cyclonedx-node-npm/actions/runs/12370296220/job/34526888496?pr=1243
could you run npm run cs-fix, @cuhland ?

Signed-off-by: Christoph Uhland <42832096+cuhland@users.noreply.github.com>
@jkowalleck
Copy link
Member

@cuhland , i will merge the feature soon, so it will be part of the next release.

next release is planned for January 2025.
Do you need a pre-release published for some early testing?

@cuhland
Copy link
Contributor Author

cuhland commented Dec 17, 2024

@cuhland , i will merge the feature soon, so it will be part of the next release.

next release is planned for January 2025. Do you need a pre-release published for some early testing?

Thanks.
I can wait for the next release.

@jkowalleck jkowalleck removed this from the v2.0.0 milestone Dec 17, 2024
@CompartMSL
Copy link

@cuhland and @jkowalleck - Thanks for integrating the --gather-license-texts option.

jkowalleck added a commit that referenced this pull request Dec 18, 2024
followup of #1243

make implementation a copy-past of existing art:
<https://github.com/CycloneDX/cyclonedx-node-yarn/blob/main/src/_helpers.ts>

this should make it easier for
CycloneDX/cyclonedx-javascript-library#1162

---------

Signed-off-by: Jan Kowalleck <jan.kowalleck@gmail.com>
@jkowalleck
Copy link
Member

this feature was released via v1.20.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: Option to add license text to SBOM result

3 participants