-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add createNamedParameter
to QueryBuilder
#11528
Conversation
I don't think |
I care about only named parameters, so this one could be removed in my opinion |
Sorry for originally suggesting adding it, I didn't see the flaw. Please remove it, and squash both commits together. |
createNamedParameter
and createPositionalParameter
to QueryBuilder
createNamedParameter
to QueryBuilder
PR ready again for review |
Thanks @norkunas ! |
Closes #9878
Replaces #9883
Cherry picked tests from #9883
Fixed review suggestions