This project is archived and is in readonly mode.
Move capture to Kernel
Reported by Marjan Krekoten' | September 16th, 2010 @ 07:42 PM
Currently capture method appears twice:
http://github.com/rails/rails/blob/master/railties/lib/rails/genera...
http://github.com/rails/rails/blob/master/railties/test/railties/en...
I think its place is in Kernel module extension in reporting. I've attached patch for this.
Comments and changes to this ticket
-
Rohit Arondekar September 17th, 2010 @ 01:43 AM
- State changed from “new” to “open”
- Importance changed from “” to “Low”
I agree! +1
-
Lake September 17th, 2010 @ 05:13 AM
+1 for staying DRY. The tests pass for me and of course the code looks better.
-
Lake September 17th, 2010 @ 05:35 AM
- Title changed from “[PATCH] Move capture to Kernel” to “Move capture to Kernel”
-
Repository September 18th, 2010 @ 07:50 PM
- State changed from “open” to “resolved”
(from [d4fa120671e989eecb71c14647dd2051d28de4a5]) Move capture to Kernel. [#5641 state:resolved] http://github.com/rails/rails/commit/d4fa120671e989eecb71c14647dd20...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Jeremy Kemper
Marjan Krekoten'
Rohit Arondekar
Santiago Pastorino
Attachments
Referenced by
-
5641
Move capture to Kernel
(from [d4fa120671e989eecb71c14647dd2051d28de4a5])
Move ca...