Skip to content
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

Update license #2915

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Update license #2915

merged 1 commit into from
Apr 4, 2024

Conversation

apinnick
Copy link
Contributor

@apinnick apinnick commented Mar 26, 2024

  • Updated the site license to CC BY-SA 4.0.
  • Updated only the year on the Red Hat license because d/s docs still refers to CC BY-SA 3.0. If this changes, I will update the RH license file.

Closes #885

Please cherry-pick my commits into:

  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (planned Satellite 6.15)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6/6.7)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • We do not accept PRs for Foreman older than 3.3.

- Updated the site license to CC BY-SA 4.0.
- Updated only the year on the Red Hat license because d/s docs still refers to CC BY-SA 3.0. If this changes, I will update the RH license file.

Signed-off-by: Avital Pinnick <[email protected]>
@apinnick apinnick force-pushed the cc-by-sa-4.0-license branch from f7cdd2c to cfda104 Compare March 26, 2024 09:28
Copy link

github-actions bot commented Mar 26, 2024

The PR preview for cfda104 is available at theforeman-foreman-documentation-preview-pr-2915.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

@ekohl ekohl mentioned this pull request Mar 28, 2024
9 tasks
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I had a look at the authors. First of all, I've taken the time to update the mailmap: #2921

With that you can easily retrieve the various authors: git log --pretty='format:%aN <%aE>' web | sort -u. That identifies b397c04 as a commit by @lumarel while all others are done by Red Hatters or @maximiliankolb.

I think technically we should get approval from @lumarel to change the license.

As for the guides themselves, the list is a bit longer. Due to privacy I'll work with @apinnick offline to come up with a plan instead copy-pasting all names here.

Copy link
Contributor

@lumarel lumarel left a comment

Choose a reason for hiding this comment

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

@ekohl it speaks very vividly for the project to even ask about such in my eyes small license changes!
Thank you for asking, and yes of course I approve 👍🏻

Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

I am OK with changing the license from "CC BY-SA 3.0" to "CC BY-SA 4.0" for my contributions.

I would like to see more people review/ACK this change.

guides/LICENSE Show resolved Hide resolved
LICENSE Show resolved Hide resolved
Copy link
Contributor

@Lennonka Lennonka left a comment

Choose a reason for hiding this comment

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

LGTM

@ekohl ekohl merged commit f524e2b into theforeman:master Apr 4, 2024
8 checks passed
@ekohl
Copy link
Member

ekohl commented Apr 4, 2024

As discussed, we don't cherry pick this to stable branches.

@apinnick apinnick deleted the cc-by-sa-4.0-license branch December 9, 2024 14:57
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.

Switch to CC BY-SA 4.0
5 participants