-
Notifications
You must be signed in to change notification settings - Fork 0
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
Disable nullable type expression for sorting #2
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you planning to merge this PR with the origin? Then maybe add also some description in the README about this feature..
The first PR I created was already targeting the real repo, so closed that again. But yes we want to merge it with the original. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would expect some updates in a CHANGELOG or README about this behavior change but other than that it looks solid to me :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
A changelog entry would be nice indeed.
…ithub.com/Inforitnl/Sieve into feature/disableSortNullableTypeExpression
MongoDB has issues with the expression created when sorting on nullable types.
So created an option to disable it.
Please also comment on things like the name of the variables/properties