-
Notifications
You must be signed in to change notification settings - Fork 1.4k
MONGOID-5702: Monkey Patch Removal: Remove __numeric__ class method from Numeric module #5746
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
base: master
Are you sure you want to change the base?
Conversation
…dule. It is now inlined into its only usage. It was previously declared @api private so assume it is safe to delete.
This should be a straightforward merge. |
@johnnyshields thanks for this one, yeah, it should be totally safe to apply, thanks to that However, I feel like this is a step backward in readability. The
|
It is now inlined into its only usage. It was previously declared @api private so assume it is safe to delete.