-
Notifications
You must be signed in to change notification settings - Fork 2
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
[INFRA] Use SPDX License Indentifier #134
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
da313ac
to
fd6b3bb
Compare
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #134 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 41 41
Lines 1361 1361
=======================================
Hits 1356 1356
Misses 5 5
☔ View full report in Codecov by Sentry. |
fd6b3bb
to
9a91b43
Compare
// MIT License: https://github.com/hideo55/cpp-HyperLogLog#license | ||
// SPDX-FileCopyrightText: 2006-2023, Knut Reinert & Freie Universität Berlin | ||
// SPDX-FileCopyrightText: 2016-2023, Knut Reinert & MPI für molekulare Genetik | ||
// SPDX-FileCopyrightText: 2013 Hideaki Ohno <hide.o.j55{at}gmail.com> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier müsste noch //SPDX-FileCopyrightText: 2022 Felix Drop
mit rein. (Jahr korrekt?)
Wenn ich mich korrekt erinnere:
Er hat das im Rahmen seiner Bachelorarbeit gemacht und nicht als Studentische Hilfskraft, daher darf sein Name da rein :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vorher war es nur die MIT license vom original, und Felix hat sich als author hinzugefügt. Weiss also nicht :D
Wurde es erstmal so lassen, und ihn fragen, ob er sich da als copyright holder mit draufsetzen will
// SPDX-FileCopyrightText: 2006-2023, Knut Reinert & Freie Universität Berlin | ||
// SPDX-FileCopyrightText: 2016-2023, Knut Reinert & MPI für molekulare Genetik | ||
// SPDX-FileCopyrightText: 2013 Hideaki Ohno <hide.o.j55{at}gmail.com> | ||
// SPDX-License-Identifier: BSD-3-Clause AND MIT |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier müste Felixs Drops name auch rein.
See also https://reuse.software/faq/
Follow-up: Update LICENSE.md
I would keep the main LICENSE.md, because it helps GitHub to display the license type. Also, it can be adapted to shortly explain how SPDX works instead of trying to explain which directory has which license.