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 #13

Merged
merged 2 commits into from
Jan 30, 2024
Merged

Update license #13

merged 2 commits into from
Jan 30, 2024

Conversation

dkoeni
Copy link
Contributor

@dkoeni dkoeni commented Jan 8, 2024

No description provided.

@dkoeni dkoeni requested a review from a team as a code owner January 8, 2024 12:14
Copy link
Contributor

@msacrea msacrea left a comment

Choose a reason for hiding this comment

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

Gemäss https://www.apache.org/licenses/LICENSE-2.0 sollte es
"Copyright 2024 SFTI" heissen nicht "Copyright 2023-2024 SFTI"

@dkoeni
Copy link
Contributor Author

dkoeni commented Jan 11, 2024

@msacrea gurndsätzlich gebe ich dir recht, aber das startdatum des copyrights scheint fast wichtiger zu sein als das aktuelle Datum. Daher gehen grosse Softwareprojekte aktuell sogar über nur das startdatum zu schreiben (siehe Google oder Microsoft). Zudem habe ich eine vorgegebene Action implementiert, die das Datum jährlich automatisch anpasst und diese passt das Datum auch zu einer Range an, daher denke ich wir sollten das auch so übernehmen und damit arbeiten.

@msacrea
Copy link
Contributor

msacrea commented Jan 30, 2024

Ok, machen wir das so.

Copy link
Contributor

@msacrea msacrea left a comment

Choose a reason for hiding this comment

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

Passt, wie besprochen.

@msacrea msacrea merged commit 1667cfa into main Jan 30, 2024
1 check passed
@msacrea msacrea deleted the license-workflow branch January 30, 2024 07:13
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