-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Fixup license #887
Fixup license #887
Conversation
cc @jasnell perhaps you could help us with this a bit? :) |
libuv is licensed for use as follows: | ||
|
||
==== | ||
Copyright (c) 2015-present libuv project contributors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2016?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arrgh, 2015-present. NVM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
life-hack to avoid the yearly silly commit 😎
LGTM |
1 similar comment
LGTM |
PR-URL: libuv#887 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Fedor Indutny <fedor@indutny.com>
PR-URL: libuv#887 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Fedor Indutny <fedor@indutny.com>
PR-URL: libuv#887 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Fedor Indutny <fedor@indutny.com>
Landed. I'll take of the boilerplate on the files when I'm on a plane or something like that :-) |
First of all: IANAL.
These changes basically reflect what we've been doing for a while.
As a follow-up, I'd like to remove all license boilerplate from the files replacing it with a "pointer" to the actual license. Example: openssl/openssl@846e33c
/cc @libuv/collaborators