Skip to content

Licenses: Clarify licenses used for this project#24

Merged
travier merged 1 commit intotrusted-execution-clusters:mainfrom
travier:main-licenses
Sep 22, 2025
Merged

Licenses: Clarify licenses used for this project#24
travier merged 1 commit intotrusted-execution-clusters:mainfrom
travier:main-licenses

Conversation

@travier
Copy link
Member

@travier travier commented Sep 16, 2025

Fixes: #8

@@ -0,0 +1,18 @@
MIT License

Copyright (c) <year> <copyright holders>
Copy link
Contributor

Choose a reason for hiding this comment

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

Fill these probably?

Copy link
Member Author

Choose a reason for hiding this comment

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

No, we should not touch those files: https://reuse.software/faq/#edit-license

Copy link
Member Author

Choose a reason for hiding this comment

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

Comment on lines +1 to +2
// SPDX-FileCopyrightText: Alice Frosi <afrosi@redhat.com>
// SPDX-FileCopyrightText: Jakob Naucke <jnaucke@redhat.com>
Copy link
Contributor

Choose a reason for hiding this comment

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

I appreciate being named, but could this get out of hand with many contributors? Other projects just do a generic <project> contributors (aka "look at the log") here

Copy link
Member Author

@travier travier Sep 16, 2025

Choose a reason for hiding this comment

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

Works for me as well. I would say that this is a starting point and we can also make this generic later.

@travier travier force-pushed the main-licenses branch 2 times, most recently from 6a80a19 to 61ab1f5 Compare September 18, 2025 14:07
@travier travier requested a review from alicefr September 18, 2025 14:08
@travier
Copy link
Member Author

travier commented Sep 18, 2025

We will need an ack from Alice before we can merge this.

@alicefr
Copy link
Contributor

alicefr commented Sep 22, 2025

/lgtm

@travier travier merged commit 3ca6e93 into trusted-execution-clusters:main Sep 22, 2025
7 checks passed
@travier travier deleted the main-licenses branch September 22, 2025 09:08
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.

Add license for the operator

3 participants