-
Notifications
You must be signed in to change notification settings - Fork 235
Create Third Party Notices.txt #603
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
Conversation
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.
Looks good! We should include this .txt file when the PSES module is distributed
@daviwil is that done in an automated way somewhere? |
Yep, that works |
|
||
UnixConsoleEcho | ||
|
||
Copyright (c) 2017 Patrick Meinecke |
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.
Shouldn't this be 2018 or at least 2017-2018? :-)
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.
I don't think you need to specify a range, copyright automatically extends quite a few years after initial creation :)
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.
Fair enough. Also, I've been told by our legal department that Copyright (c)
is redundant. You only need one of: Copyright
or (c)
or ©
. RE ranges and copyright update, I was told it should be updated whenever the associated source file (or copyrighted artifact) is updated so in our older code-base you see a combination of comma separate years and ranges. Because of the "copyright extends past the last date for some (large) number of years" we got our legal dept to buy off on us not putting copyright years in our source code. It got to be too much of a PITA to keep it correct and besides, devs often forget to update copyright notices in source code anyway. :-)
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.
Pretty sure he was just copying my notice :P
If it makes anything easier I can just remove the copyright from the license. Pretty sure it was just a part of whatever template I copied it from :)
Either way I can guarantee I'll never update it :P
|
||
UnixConsoleEcho | ||
|
||
Copyright (c) 2017 Patrick Meinecke |
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.
Fair enough. Also, I've been told by our legal department that Copyright (c)
is redundant. You only need one of: Copyright
or (c)
or ©
. RE ranges and copyright update, I was told it should be updated whenever the associated source file (or copyrighted artifact) is updated so in our older code-base you see a combination of comma separate years and ranges. Because of the "copyright extends past the last date for some (large) number of years" we got our legal dept to buy off on us not putting copyright years in our source code. It got to be too much of a PITA to keep it correct and besides, devs often forget to update copyright notices in source code anyway. :-)
It's fine. Copyright notices get me going having gone a few rounds with our legal folks. Go ahead and merge it. |
…owerShell#1281) Fix PowerShell#603 where system has ipv6 addresses disabled. This is also more consistent with the PSES TCP listener which always [IPAddress]::Loopback as the IP address.
For legal purposes.
for reference: https://github.com/PowerShell/vscode-powershell/blob/master/Third%20Party%20Notices.txt