Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Remove deprecated 'rake/rdoctask', replace with new invocation. #61

Merged
merged 2 commits into from Jun 14, 2015

Conversation

Projects
None yet
5 participants
Contributor

mipearson commented Dec 16, 2012

Required for rake 10.

Member

jeremy commented Dec 16, 2012

Perhaps this deserves adding a dev dependency on rake to the gemspec?

Owner

raggi commented Dec 17, 2012

You mean rdoc and rake...

Contributor

mpalmer commented Oct 31, 2014

Yes, oh yes, oh ghods yes.

@mpalmer mpalmer added the mpalmer-ok label Oct 31, 2014

Contributor

jjb commented Nov 2, 2014

I'm not familiar with this issue but looks legit.

Should it be in a major release because there is a dependency change?

Contributor

jjb commented Nov 2, 2014

or at this point can it be considered a bug...

Contributor

mpalmer commented Nov 2, 2014

A dependency change doesn't require a major release change, IMO. Nothing in the user's code has to change.

Contributor

jjb commented Nov 2, 2014

k

Contributor

jjb commented Nov 2, 2014

happy to merge then

let's wait 48 hours to see if any of the other folks chime in

@mpalmer mpalmer added a commit that referenced this pull request Jun 14, 2015

@mpalmer mpalmer Merge pull request #61 from mipearson/master
Remove deprecated 'rake/rdoctask', replace with new invocation.
4ce89d0

@mpalmer mpalmer merged commit 4ce89d0 into rack:master Jun 14, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment