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

chore: update author in copyright headers #68

Merged
merged 1 commit into from
Nov 8, 2023
Merged

Conversation

JKRhb
Copy link
Member

@JKRhb JKRhb commented Nov 8, 2023

Resolves #67.

@JKRhb
Copy link
Member Author

JKRhb commented Nov 8, 2023

CC @falko17

@JKRhb JKRhb merged commit 1a8bbfc into main Nov 8, 2023
4 checks passed
@JKRhb JKRhb deleted the update-copyright-headers branch November 8, 2023 14:48
@egekorkan
Copy link
Member

By the way, why not use 2023 everywhere?

@JKRhb
Copy link
Member Author

JKRhb commented Nov 10, 2023

By the way, why not use 2023 everywhere?

I personally like to only use the first year the respective source file was created (as this way you don't need to continuously update the year). However, we could of course also use 2023 and/or use a similar strategy as in node-wot.

@danielpeintner
Copy link
Member

I don't have a strong preference (do we need one for thingweb projects?).

Anyhow, I agree that keeping the year up-to-date is cumbersome, especially if we update the year every time changes to a file are made (as done in node-wot).
Usually I like to see both "year"s, first the year once a file has been first created and second once it was recently updated ... but that would lead to a lot of efforts that might not be worth it.

In node-wot we use a simple Java code that updates the copyrights...

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.

Update author in copyright headers
4 participants