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 Confio GmbH copyright years in NOTICE #2251

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

webmaster128
Copy link
Member

No description provided.

@@ -1,7 +1,7 @@
Copyright 2019 Jehan Tremback
Copyright 2019-2021 Confio OÜ
Copyright 2019-2020 Simon Warta
Copyright 2021 Confio GmbH
Copyright 2021,2022,2023,2024 Confio GmbH
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe 2021-2024 would be simpler?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, no strong opinion. I once heard a guy saying it is better to list the years of actual authorship such that years without contributions are skipped. This is why I did it that way.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yeah, that makes sense, but I don't see us stopping for a year inbetween. :D
And then the list will get very long (hopefully).

Copy link
Collaborator

Choose a reason for hiding this comment

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

But any way is fine for me. We can still switch if it gets out of hand.

@webmaster128 webmaster128 merged commit 9f623aa into main Sep 12, 2024
32 of 33 checks passed
@webmaster128 webmaster128 deleted the webmaster128-patch-1 branch September 12, 2024 09:18
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