-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Remove special behavior for rake #9245
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
Conversation
|
I also encountered this code a few weeks ago and couln't figure out why it was needed. FWIW it was added in 20cdf22b844 but I can't find the original rubyforge discussion that led to this change. |
|
@JasonLunn Can you add test for this? I couldn't confirm this change works correctly. |
|
@matthewd Found the rubyforge discussion. This is the reason this patch was applied
and the patch was proposed here https://web.archive.org/web/20140515164338/http://rubyforge.org/pipermail/rubygems-developers/2006-July/002056.html |
|
Regression test added. |
|
@hsbt - can you re-run tests now that the linter has been appeased? |
|
Extra whitespace fixed and re-based, ready for re-testing |
|
Thanks, @hsbt! |
Fixes #9244