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.

inline exception configuration

#19

would be nice to be able to disable smells for specific bits of code instead of using the config file. for example...

def my_method #:nosmell[utility_function] ... end

Reported by S. Brent Faulkner · February 18th, 2009 @ 06:28 PM

State: resolved
Milestone: none
Assigned to: Kevin Rutherford Kevin Rutherford

Activity

  1. S. Brent Faulkner
    S. Brent Faulkner

    better formatting for example...

    
def my_method #:nosmell[utility_function]
  ...
end

    February 18th, 2009 @ 06:29 PM

  2. Kevin Rutherford
    Kevin Rutherford

    Agree this would be nicer.

    Right now, reek doesn't see the source code -- and therefore it can't see comments either. The config file was the simplest alternative I could deliver without a serious rewrite.

    To help me decide what to do next, could you give an example of a method that is incorrectly (or unreasonably) flagged as a UtilityFunction?

    February 26th, 2009 @ 10:20 PM

  3. Kevin Rutherford
    Kevin Rutherford
    • Assigned user set to Kevin Rutherford

    February 26th, 2009 @ 10:20 PM

  4. Kevin Rutherford
    Kevin Rutherford
    • State changed from new to resolved

    This ticket has been moved to http://github.com/kevinrutherfor.../17

    April 23rd, 2009 @ 04:01 PM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.