-
Notifications
You must be signed in to change notification settings - Fork 17
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
Work not only in the public schema #76
Conversation
@vgarvardt please update with master when you have the time 🙏 |
@lucasmdrs rebased |
the tests seem to be a bit flaky, I'll take a look at that later 🙁 |
It seems that unit test is failing due to some hardcoded timing condition. |
Tried to reproduce the problem locally, but all the 50 runs I did were successful. |
I managed to reproduce the case locally, decided to fix by changing the way the assertions are being made |
Cool, thank you! Will rebase my branch after #78 is merged. |
rebased |
@lucasmdrs can you pls release new version with the fixed and updates lately merged |
Currently
public
schema is hardcoded that prevents some functionality from working correctly when non-publis schema is being used.