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

chore: Update license #600

Merged
merged 6 commits into from
Jul 3, 2019
Merged

chore: Update license #600

merged 6 commits into from
Jul 3, 2019

Conversation

jeeyyy
Copy link
Collaborator

@jeeyyy jeeyyy commented Jun 12, 2019

@jeeyyy jeeyyy added the Chore label Jun 12, 2019
nitedog
nitedog previously approved these changes Jun 12, 2019
Copy link
Collaborator

@nitedog nitedog left a comment

Choose a reason for hiding this comment

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

Do we need to copy the license text or is it sufficient to provide the license name and link? W3C has persistent links, so there should be no problem in my little understanding of licensing.

@jeeyyy jeeyyy self-assigned this Jun 12, 2019
pages/license.md Outdated Show resolved Hide resolved
pages/license.md Outdated Show resolved Hide resolved
@jeeyyy
Copy link
Collaborator Author

jeeyyy commented Jun 13, 2019

@nitedog this is not just a link, I have copied over the text.

@jeeyyy jeeyyy requested a review from nitedog June 13, 2019 10:19
@nitedog
Copy link
Collaborator

nitedog commented Jun 14, 2019

@JKODU sorry if I was not clear -- I'm proposing to simply link to the license rather than to copy and paste the text. What is the reason for copying the license text? This creates confusion and headache in my view (people have to actually compare if it is the same license text etc.).

@jeeyyy
Copy link
Collaborator Author

jeeyyy commented Jul 3, 2019

@nitedog

The reason for copy/ pasting the text is that, if and when the original license changes, it does not change the license on this repo automagically.

Any license changes will be done via a PR, which will help notify and communicate of such changes.

@jeeyyy jeeyyy merged commit 8840f83 into develop Jul 3, 2019
@jeeyyy jeeyyy deleted the update-license branch July 3, 2019 11:36
@jeeyyy jeeyyy mentioned this pull request Jul 3, 2019
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.

3 participants