Skip to content

[MRG] License update (#10228)#10229

Merged
agramfort merged 1 commit intomne-tools:mainfrom
stevemats:license-update
Jan 20, 2022
Merged

[MRG] License update (#10228)#10229
agramfort merged 1 commit intomne-tools:mainfrom
stevemats:license-update

Conversation

@stevemats
Copy link
Copy Markdown
Contributor

Reference issue

What does this implement/fix?

  • Updates License dates on README.rst
  • Updates License dates on LICENSE file

@cbrnr
Copy link
Copy Markdown
Contributor

cbrnr commented Jan 20, 2022

We could also think about not updating the current year because it doesn't seem to be necessary: https://hynek.me/til/copyright-years/

@stevemats
Copy link
Copy Markdown
Contributor Author

We could also think about not updating the current year because it doesn't seem to be necessary: https://hynek.me/til/copyright-years/

Thanks for sharing this. My original thought was that derivative works can appear and use your copyright in 2022 and say that you didn't have/claim the copyright this year hence, giving permission for anyone to use the copyright with no claim.

@cbrnr
Copy link
Copy Markdown
Contributor

cbrnr commented Jan 20, 2022

Yes, that's apparently not how it works.

@wmvanvliet
Copy link
Copy Markdown
Contributor

Still, seems neater to me to have it updated.

@cbrnr
Copy link
Copy Markdown
Contributor

cbrnr commented Jan 20, 2022

Still, seems neater to me to have it updated.

As opposed to not updating? Sure! As opposed to not including any years? Nope, I'd prefer that. But I'm fine with either solution because at least we don't include this in all of our files.

@agramfort agramfort closed this Jan 20, 2022
@agramfort agramfort reopened this Jan 20, 2022
@agramfort agramfort merged commit 4705183 into mne-tools:main Jan 20, 2022
@agramfort
Copy link
Copy Markdown
Member

thx @stevemats

@stevemats stevemats deleted the license-update branch January 20, 2022 20:49
@sappelhoff sappelhoff mentioned this pull request Jan 21, 2022
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.

4 participants