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
Support for strict tables #1288
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.
Thank you @jcavar :-) This looks very good.
During the refactoring, one test was lost, and I think we should restore it.
Also, skip the test for old SQLCipher version.
It's better to quickly catch failures to support SQLCipher 3.
I took the liberty to apply the necessary fixes. Also, the test is now skipped for SQLCipher 3. |
OK, tests pass 👍 Thank you very much @jcavar, I'll prepare a release shortly. |
Hey @groue, sorry, I am little slower. Indeed, I've probably deleted the most useful part of the strict tables test. Thanks for fixing it for me and thank you for your great work on this library! |
You're welcome, and happy GRDB :-) |
Resolves #1287
Pull Request Checklist
development
branch.make smokeTest
terminal command runs without failure.