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 LICENSE file #22

Merged
merged 3 commits into from
Nov 2, 2016
Merged

Add LICENSE file #22

merged 3 commits into from
Nov 2, 2016

Conversation

Fale
Copy link
Contributor

@Fale Fale commented Oct 31, 2016

No description provided.

@coveralls
Copy link

coveralls commented Oct 31, 2016

Coverage Status

Coverage remained the same at 92.206% when pulling 856cf92 on Fale:patch-1 into e42064b on arthepsy:master.

Copy link
Owner

@arthepsy arthepsy left a comment

Choose a reason for hiding this comment

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

Please, include the header line "The MIT License (MIT)" and wrap it the same as in source code or http://choosealicense.com/licenses/mit/

@Fale
Copy link
Contributor Author

Fale commented Nov 1, 2016

Now is exactly like the choosealicense one. Is it ok?

@coveralls
Copy link

coveralls commented Nov 1, 2016

Coverage Status

Coverage remained the same at 92.206% when pulling 06d76f6 on Fale:patch-1 into e42064b on arthepsy:master.

Copy link
Owner

@arthepsy arthepsy left a comment

Choose a reason for hiding this comment

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

Sorry, not really. First line should be The MIT License (MIT).

And the text must wrap (with newlines). Currently, those sentences are just big lines. If You look at licence in source code or at URL I provided, You'll see that sentences are wrapped to be of (column) width 80. To ease the pain, just copy/paste the rest after copyright 😃

@Fale
Copy link
Contributor Author

Fale commented Nov 2, 2016

Better now?

@coveralls
Copy link

coveralls commented Nov 2, 2016

Coverage Status

Coverage remained the same at 92.206% when pulling 04d8e2e on Fale:patch-1 into e42064b on arthepsy:master.

@arthepsy arthepsy merged commit 22b671e into arthepsy:master Nov 2, 2016
@arthepsy
Copy link
Owner

arthepsy commented Nov 2, 2016

Perfect. Thanks!

arthepsy pushed a commit that referenced this pull request Nov 2, 2016
@jtesta jtesta mentioned this pull request Nov 8, 2020
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