-
-
Notifications
You must be signed in to change notification settings - Fork 233
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
feat: add Join to UpdateQuery #908
Conversation
1436b9b
to
6b85689
Compare
@svanharmelen looks good 👍 Could you please update query_test.go with some example? |
6b85689
to
343378d
Compare
Improves readability (IMHO) so thought to implement it.
343378d
to
da3a04e
Compare
@vmihailenco added a test (and made sure it succeeds)... Let me know if you need anything else |
@vmihailenco anything else you need from my side? Thanks! |
Anything I can do to get this one merged and released? Would be nice if we can switch back to this package and not have to rely on a |
Can we please merge this @vmihailenco? Anything more needed from my side? Thanks! |
@svanharmelen sorry for the delay 👍 |
Thanks @vmihailenco! Only thing left would be a new tag that includes these changes... Any plan for that already? |
Thanks for the release @vmihailenco 🎉 |
@svanharmelen thanks for the contribution 👍 |
Of course I can remove the second commit if that would block merging this PR!
Fixes #907