-
Notifications
You must be signed in to change notification settings - Fork 33
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
Use separated headers #1142
Use separated headers #1142
Conversation
fa5e53d
to
ea5a2df
Compare
Is it better @bclement-ocp ? |
I meant that the year should ideally depend on the file. If we can't do that, keeping the start of the project date (2013) everywhere is better. |
So you want a copyright of the form |
I don't really see the point of this format: the main point here is to not claim copyright before the code was actually written. So ideally we would have If that is technically challenging with the tools we have, |
Use separated headers to distinguish our contributions.
ea5a2df
to
da4450a
Compare
I removed years in all the headers as we decided during the last dev meeting. |
I think this is correct wrt the files changed. I understood we chose the option of keeping "2013 - 2024" everywhere, but dropping the years is fine by me should be fine (this is what curl did recently, and code from big companies such as VScode do not have copyright years) and should simplify our life. Would prefer an ack from @lefessan or @Ninjapouet before merging. |
I think we can merge. It is really easy to rollback it with |
Fabrice agreed so we can merge @bclement-ocp |
Use separated headers to distinguish our contributions.