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
Two test failures when ujson installed #262
Comments
I've now fixed this by skipping these tests when |
The problem has not been solved by 4e6f849 If ujson is installed, using tinydb as documented breaks. i.e. use of "Deprecating" it is not nice. The user isnt explicitly trying to use it with |
You're right that I was a little quick to add a deprecation warning. I think it wasn't the best idea to add this auto-detection mechanism in the first place. Maybe it would be better to raise a warning that the behavior of In the long term I think we'd need to release a breaking release of TinyDB that removes |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reopen this if needed. Thank you for your contributions ❤️ |
Still an issue. |
This should be fixed with TinyDB v4 where |
One still can write a custom storage that uses the |
Thanks. |
When ujson is installed, there are two errors in the test suite.
Builds at https://build.opensuse.org/package/show/home:jayvdb:py-new/python-tinydb
The text was updated successfully, but these errors were encountered: