having the old method around can be nice during debugging since it shows
nicely in the backtrace + can be called,
but either way is fine


On Thu, Jan 2, 2014 at 2:49 PM, Steven Davidovitz
<notifications@github.com>wrote:

> In lib/logcast.rb:
>
> >
> >    def self.engage!
> > -    logger = if defined?(Rails.logger)
> > -      Rails.logger
> > -    elsif defined?(RAILS_DEFAULT_LOGGER)
> > -      RAILS_DEFAULT_LOGGER
> > -    else
> > -      ActiveRecord::Base.logger
> > -    end
> > -
> > -    logger = logger.instance_variable_get(:@logger) if logger.class.name == "ActiveSupport::TaggedLogging"
> > +    original = Rails.method(:logger=)
>
> I don't really want the old method laying around, but they're essentially
> the same
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zendesk/logcast/pull/3/files#r8625605>
> .
>