Skip to content

Add MIT license indicator to the gemspec.#23

Merged
jch merged 2 commits into
gjtorikian:masterfrom
nbibler:license_gemspec
Nov 29, 2012
Merged

Add MIT license indicator to the gemspec.#23
jch merged 2 commits into
gjtorikian:masterfrom
nbibler:license_gemspec

Conversation

@nbibler

@nbibler nbibler commented Nov 29, 2012

Copy link
Copy Markdown
Contributor

This pull simply adds the MIT license indicator to the gemspec. This benefits developers by allowing Rubygems.org to display the proper license on the gem page and gives automated license checkers the ability to determine the intended license without looking into the repository.

Comment thread html-pipeline.gemspec Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please use double-quotes like the rest of this file. :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure. I went with single since the add_dependency calls were all single... but I can change it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yup, I'll fix the other ones if you get the one in this PR. Thanks!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

A new commit has been added to this PR with the modified quotes.

@jch

jch commented Nov 29, 2012

Copy link
Copy Markdown
Contributor

Thanks! good pull.

jch added a commit that referenced this pull request Nov 29, 2012
Add MIT license indicator to the gemspec.
@jch jch merged commit 41ed791 into gjtorikian:master Nov 29, 2012
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