Skip to content

Add license target to project - #395

Merged
tinder-maxwellelliott merged 7 commits into
Tinder:masterfrom
ViggedalZenseact:add_license_target_to_project
Aug 10, 2026
Merged

Add license target to project#395
tinder-maxwellelliott merged 7 commits into
Tinder:masterfrom
ViggedalZenseact:add_license_target_to_project

Conversation

@ViggedalZenseact

Copy link
Copy Markdown
Contributor

This makes the license observable through Bazel and facilitates for consumers of this project the automated building of an SBOM.

This makes the license observable through Bazel and facilitates for
consumers of this project the automated building of an SBOM.

@tinder-maxwellelliott tinder-maxwellelliott left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for doing this

Comment thread BUILD
Comment thread BUILD
Adding a CI step to build the license target to ensure it's kept intact.
Modifying generate-readme tool to keep the package_version of the license
target in sync with the current version of the project.
There was copy paste mistake in specifying the license kind.
@tinder-maxwellelliott

Copy link
Copy Markdown
Collaborator

Is there anything to do here on the BCR side of this? Ideally we have everything wired up for that now

@ViggedalZenseact

Copy link
Copy Markdown
Contributor Author

Sorry for being slow to reply. I went on vacation for a few weeks and haven't had time to check back in.
I don't think there's anything to do on the BCR side but that's a good question, I would like to double check before continuing. I will be back working on this in three weeks.

@ViggedalZenseact

Copy link
Copy Markdown
Contributor Author

As far as I can tell there is nothing that needs to be done alos on the BCR side.

@tinder-maxwellelliott
tinder-maxwellelliott merged commit 9e82cab into Tinder:master Aug 10, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants