-
Notifications
You must be signed in to change notification settings - Fork 22
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
Runtime error on Ruby 2, Rails 4 #11
Comments
Thanks for pointing it out! A pull request would be awesome, otherwise I'll try to get to it when I can but I haven't had much time to play with Rails 4 yet. |
http://edgeguides.rubyonrails.org/4_0_release_notes.html#active-support-deprecations
I simply replaced all gem 'better_logging', github: 'snow/better_logging' in rails4 projects and origin gem in rails3 projects. And this folk is only intend for temporary use till the origin gem release a version that supports rails 4. sorry for my broken Englisth. |
Thanks, I'll try to find the time to do it in a conditional way. |
and, in lib/better_logging/better_logging.rb, ln 32, should use |
Thanks! Update for Rails 4 coming as soon as I have time. |
See pauldowman/better_logging#11 (comment) That was **8 months** ago... Ouch
See pauldowman/better_logging#11 (comment) That was **8 months** ago... Ouch
See pauldowman/better_logging#11 (comment) That was **8 months** ago... Ouch
Actually, when I switch to this other fork (on rails 4), the output is now empty, like this:
|
Sorry, I'm not maintaining (or even using) this any more, but check @foodforarabbit's fork: |
A bit ahead of the curve, perhaps, but I'm getting an error at runtime when I boot up a Rails 4 (beta1) server; evidently
ActiveSupport::BufferedLogger
is no more.If/when I have time, I may submit a pull request to see if I can fix this, unless someone else beats me to it.
The text was updated successfully, but these errors were encountered: