-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
changed all instances of all.map to pluck #5509
Conversation
Can you also change in other files? |
I think that we don't need to change it in tests. |
70a19b7 this got merged. If we have to change let's do it in one shot. |
This change looks weird to me because |
Yeah, the semantics are different but pluck will be faster than an all.map. So, it makes sense to change it, right? |
IMO we shouldn't change semantics in tests. |
prevent returning a negative zero from number_with_precision
Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header.
Closing this. Please reopen this PR with proper commits in it. I think it's somehow messed up with other commits. Thanks :-) |
No description provided.