Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

This project is archived and is in readonly mode.

Move capture to Kernel

#5641

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.

Reported by Marjan Krekoten' · September 16th, 2010 @ 07:42 PM

State: resolved
Milestone: none
Assigned to: Santiago Pastorino Santiago Pastorino
Importance: Low

Activity

  1. Rohit Arondekar
    Rohit Arondekar
    • State changed from new to open
    • Importance changed from to Low

    I agree! +1

    September 17th, 2010 @ 01:43 AM

  2. Lake
    Lake

    +1 for staying DRY. The tests pass for me and of course the code looks better.

    September 17th, 2010 @ 05:13 AM

  3. Lake
    Lake
    • Title changed from [PATCH] Move capture to Kernel to Move capture to Kernel

    September 17th, 2010 @ 05:35 AM

  4. Repository
    Repository
    • State changed from open to resolved

    September 18th, 2010 @ 07:50 PM