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 legal docs (TRG7) #476

Merged
merged 2 commits into from
May 7, 2024
Merged

Update legal docs (TRG7) #476

merged 2 commits into from
May 7, 2024

Conversation

AngelikaWittek
Copy link
Contributor

@AngelikaWittek AngelikaWittek commented Apr 23, 2024

Description

Updates due to (upcoming) changes in TRG 7:

  • added Terms of Use in CONTRIBUTING file
  • extended NOTICE file with a notice for the CC-BY-4.0 license
  • added the CC-BY-4.0 license to the root repository as default license for non-code artifacts

Pre-review checks

Not necessary for the changes.

Copy link
Contributor

@FaGru3n FaGru3n left a comment

Choose a reason for hiding this comment

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

Thx @AngelikaWittek, LGTM

@SebastianBezold
Copy link
Contributor

Hi @FaGru3n, since the description mentions UPCOMING changes, I think @AngelikaWittek should be the one to move the PR from "Draft" to "Ready for review".

@FaGru3n
Copy link
Contributor

FaGru3n commented Apr 23, 2024

Hi @FaGru3n, since the description mentions UPCOMING changes, I think @AngelikaWittek should be the one to move the PR from "Draft" to "Ready for review".

my bad sorry for that, but since all is written already we could allign on the date when we will merge the appoved PR... and added everyone within system team to inform everyone what will be the upcomming changes.

converted it back to Draft

@AngelikaWittek
Copy link
Contributor Author

AngelikaWittek commented Apr 23, 2024

Yes, it was intentionally put in the DRAFT state. Please do NOT merge!
There has to be some more work on the TRG7s and communicated with the committers.
I just used this repo with this PR as an example to demonstrate upcoming changes.

@FaGru3n FaGru3n marked this pull request as draft April 23, 2024 12:30
@AngelikaWittek AngelikaWittek marked this pull request as ready for review May 7, 2024 08:19
Copy link
Contributor

@SebastianBezold SebastianBezold left a comment

Choose a reason for hiding this comment

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

LGTM. Also good template for others to follow

This was referenced Jun 3, 2024
@FaGru3n FaGru3n deleted the update_legal_docs branch August 1, 2024 07:43
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.

3 participants