Remove some redundant calls and make the code more readable.
Make Sprockets::Helpers::RailsHelper#debug_assets? more readable
Seems good. I wonder though, do people call with debug_assets = false? Why are we explicitly testing for a value?
debug_assets = false
@tenderlove I see no reason for it. I'll be more than happy to simply check for the existance of the parameter - should I add it as a separate pull request?
Yes, please send a new pull request. If the tests pass with merely checking for the key, then I'll apply. :-)
EDIT: not just if hash.key? though, we should probably just do if hash[:debug_assets]
@tenderlove: I've added a pull request per your comment: #2695.