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

Gemspec updated #109

Closed
wants to merge 4 commits into from
Closed

Gemspec updated #109

wants to merge 4 commits into from

Conversation

iGEL
Copy link
Contributor

@iGEL iGEL commented Aug 24, 2014

There are multiple PRs to update the Gemspec, but they missed that it's generated with Thor. Here an updated version:

Also, it declares all development dependencies (including the missing thor & git) in the gemspec, not the Gemfile & removed the like to the late rubyforge project.

iGEL added 4 commits August 24, 2014 22:46
It's recommended to not include the Gemfile.lock in a project, see
http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/

Also, I declared the development dependencies in the gemspec and just
refer to that from the Gemfile.

Lastly, I declared two so far undeclared dependencies: thor & git
@whatyouhide
Copy link

👍

@kkirsche
Copy link

Partial dupe of #72

perlun pushed a commit that referenced this pull request Apr 21, 2017
@junaruga
Copy link
Contributor

@iGEL Thank you for your pull-request.
As information, rack-test.gemspec license part of this PR was merged today.
fed7b5d

We will check other parts in your PR too to continue.

@junaruga
Copy link
Contributor

junaruga commented May 30, 2017

@iGEL this PR's other parts were also merged at #181 .
So, I would close this ticket.
If you have something, let me know. Thank you.

@junaruga junaruga closed this May 30, 2017
@iGEL iGEL deleted the gemspec branch March 12, 2018 10:00
alex-damian-negru pushed a commit to alex-damian-negru/rack-test that referenced this pull request Apr 5, 2021
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.

4 participants