Skip to content

Add license statement #52

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

Merged
merged 9 commits into from
Jul 21, 2015
Merged

Add license statement #52

merged 9 commits into from
Jul 21, 2015

Conversation

PragTob
Copy link
Member

@PragTob PragTob commented Mar 15, 2015

Includes German and English version.

Missing: Spani/Polish also a review

@PragTob
Copy link
Member Author

PragTob commented Mar 15, 2015

@chastell @xamebax @eljojo @myabc could you please edit/add the corresponding versions in your respective language? Thanks <3 I think PRs against this branch should be fine, or just straight up add it to this branch, if you don't have commit rights and want them please let me know :)

@chastell
Copy link
Collaborator

Done, apologies for taking so much time. I also fixed some rogue Unicode characters in the English version.

BTW: Was there a conscious decision to use American English for the licence? ;) 🇬🇧

@skade
Copy link
Contributor

skade commented Mar 25, 2015

I don't think so and we usually standardise on spelling.

@PragTob
Copy link
Member Author

PragTob commented Mar 25, 2015

Thanks Piotr!

There wasn't - I think it's just our American influenced culture. I won't
have an AE and BE version though, that's just confusing.

sent from my mobile - sorry for the brevity

www.pragtob.info
On Mar 25, 2015 10:50 AM, "Piotr Szotkowski" [email protected]
wrote:

Done, apologies for taking so much time. I also fixed some rogue Unicode
characters in the English version.

BTW: Was there a conscious decision to use American English for the
licence? ;)


Reply to this email directly or view it on GitHub
#52 (comment)
.

@chastell
Copy link
Collaborator

Ok, thanks for the info. The rest of the CoC is in AmE, so let’s keep ‘license’ for this PR and come back to the Serious Issue of Spellink separately when there are no more pressing issues. :)

@PragTob
Copy link
Member Author

PragTob commented Mar 25, 2015

@eljojo @myabc some Spanish would be highly appreciated :)

@eljojo
Copy link
Member

eljojo commented Mar 26, 2015

@PragTob just updated the Spanish version. Maybe @catalanojuan could also share his opinion, but I think it's ok. (at least good enough).

@eljojo
Copy link
Member

eljojo commented Mar 26, 2015

I guess this is ready to merge :shipit:

@catalanojuan
Copy link
Contributor

lgtm! Thanks for looping me in!

@PragTob
Copy link
Member Author

PragTob commented Mar 26, 2015

now I just notice that we are missing our brand new french version... @lucaspinto @fannivuniconnu @gregkare @thegcat

@lucaspinto
Copy link
Contributor

I slept on this, sorry. I'll be taking care of it during Easter.

@PragTob
Copy link
Member Author

PragTob commented Apr 6, 2015

Thanks lucas :)

@osg
Copy link

osg commented Apr 28, 2015

@PragTob Nice work! I assume the license will be international. :) http://creativecommons.org/licenses/by-sa/4.0/

@PragTob
Copy link
Member Author

PragTob commented Jun 7, 2015

@lucaspinto @fannivuniconnu @gregkare @thegcat A small french version would be highly appreciated so we can merge this in :)

@PragTob
Copy link
Member Author

PragTob commented Jun 7, 2015

@opensourcegrrrl yes that is the license I'm referring to, maybe we should link to it

@thegcat
Copy link

thegcat commented Jun 7, 2015

@PragTob see #64

@thegcat
Copy link

thegcat commented Jun 7, 2015

@PragTob and regarding the license: I'd very much be in favor of linking to it with the corresponding locale, as I wouldn't have known where exactly to look either.

@PragTob
Copy link
Member Author

PragTob commented Jun 7, 2015

first thanks, secondly we can probably work that in in the next revision :)

PragTob added a commit that referenced this pull request Jul 21, 2015
@PragTob PragTob merged commit 6e561a5 into master Jul 21, 2015
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.

8 participants