Build: Remove dates from copyright notice #1983
Conversation
Gruntfile.js
Outdated
There was a problem hiding this comment.
Why the change from (c) to Copyright? You didn't do the same in the Sizzle PR: https://github.com/jquery/sizzle/pull/307/files
This is the banner put in the minified files, e.g. https://code.jquery.com/jquery-2.1.3.min.js
There was a problem hiding this comment.
I'd leave | at the end, it separates the copyright info from the URL to the license (see the linked minified file for example).
There was a problem hiding this comment.
@mzgol Thanks for the comments. I'll make the changes now.
|
LGTM |
There was a problem hiding this comment.
In the intro file you also dropped the Inc
There was a problem hiding this comment.
+1, why this is being dropped btw?
There was a problem hiding this comment.
We don't need them, according to lawyers. So removing the dates makes it easier to maintain! 😃
There was a problem hiding this comment.
I'll remove the Inc. here for consistency.
I'm not too sure about the change in Gruntfile.js, so please let me know if you would like it to be different, e,g, whether the jquery.org/license should be on the same line as the Copyright notice, whether it should be https://jquery.org/ or https://jquery.org/license also, or anything else.