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

Add a "License" file to the repository #95

Closed
wants to merge 1 commit into from
Closed

Add a "License" file to the repository #95

wants to merge 1 commit into from

Conversation

ingydotnet
Copy link
Member

This is simply the Copyright text that appears at the front of the spec.

@ingydotnet ingydotnet requested a review from a team as a code owner July 31, 2021 14:07
@ingydotnet ingydotnet force-pushed the license branch 3 times, most recently from 2020f83 to f275cf5 Compare August 5, 2021 14:41
Comment on lines +3 to +4

This document may be freely copied, provided it is not modified.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This document may be freely copied, provided it is not modified.

I don't think we should include this here, as outside of the compiled spec it's unclear what "this document" really means.

At a first glance, it would seem that we might want to use CC-BY-ND as a license, but I'm not sure if there are considerations that I'm not taking into account here.

@ingydotnet ingydotnet force-pushed the license branch 2 times, most recently from 1dff2c3 to f8df286 Compare August 7, 2021 20:21
@ingydotnet ingydotnet force-pushed the license branch 2 times, most recently from f53cb5c to 2282ce6 Compare August 28, 2021 12:27
@Thom1729 Thom1729 added the Spec Change Any change to the specification itself label Sep 3, 2021
@ingydotnet ingydotnet marked this pull request as draft September 4, 2021 21:11
@ingydotnet
Copy link
Member Author

I'm closing this for now.

@jcflack
Copy link

jcflack commented Feb 28, 2022

I'm adding a comment to this issue just to connect it to yaml/yaml-reference-parser#4, given that the issues are kind of inseparably connected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Spec Change Any change to the specification itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants