Skip to content
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

prevent crashes when testing without defined @logger #3712

Closed
wants to merge 3 commits into from
Closed

prevent crashes when testing without defined @logger #3712

wants to merge 3 commits into from

Conversation

slbug
Copy link
Contributor

@slbug slbug commented Nov 21, 2011

No description provided.

@danpal
Copy link

danpal commented Dec 6, 2011

Please accept.....I keep hitting this bug

@gregwebs
Copy link

I ran into this bug today also.

@reneedv
Copy link

reneedv commented Jan 21, 2012

I just ran into this issue too.

@mkoby
Copy link

mkoby commented Feb 3, 2012

I ran into this as well and spent two days tracking down the issue. For me I was using ActiveRecord outside of Rails, and doing a

ActiveRecord::Base.logger = Logger.new(STDERR)

got me past the error, so now I'll just make sure we're actually handling logging in my app.

@carlosantoniodasilva
Copy link
Member

I think this is already fixed in branches, for instance 3-2 af02291 and master b5c939d.

@josevalim josevalim closed this Mar 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants