-
Notifications
You must be signed in to change notification settings - Fork 157
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
No longer make copyright lines stringly typed #947
Labels
enhancement
New feature or request
Comments
This was referenced Apr 27, 2024
This was referenced Jun 3, 2024
I like this. Two changes I’d like to propose though:
|
@silverhook all fields (except name) will be optional :) for the formats, see the
|
Personally I’d limit it down to:
Maaaybeee include:
…but you know my thoughts on this ;) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Presently, copyright lines are stringly typed in the tool. The whole thing
SPDX-FileCopyrightText: 2016, 2018--2020, 2023--present Carmen Bianca BAKKER <carmenbianca@fsfe.org>
is one string.I think it might make sense to write some kind of
CopyrightLine
class that can parse elements from such strings. For example, the above properties of the above example:SPDX-FileCopyrightText
2016
,2018--2020
, and2023--present
. Or, if that all is too over-engineered, just the string2016, 2018--2020, 2023--present
Carmen Bianca BAKKER
carmenbianca@fsfe.org
If implemented well, this could heavily simplify date-range merging, which is currently spaghetti code.
The text was updated successfully, but these errors were encountered: