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.

Sign up for GitHub

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

Jump to bottom

Add check for temporary database path #4662

Merged

AlecKazakova merged 1 commit into cashapp:master from 05nelsonm:issue/3241-native-temporary

Oct 18, 2023

Contributor

05nelsonm commented Sep 25, 2023

This PR adds to the NativeSqliteDriver a check for use of a "temporary" database (i.e. an empty path).

NOTE: I added to the EphemeralTest an override to skip the Type.NAMED test for the time being (and commented on it) because of #3241 (comment)


          Add check for temporary database path

64b2b50

AlecKazakova approved these changes

View reviewed changes

AlecKazakova merged commit c7c941c into cashapp:master

7 checks passed

lnhrdt mentioned this pull request

Draft a 2.0.1 changelog #4781

Merged

hfhbd pushed a commit that referenced this pull request


          Add check for temporary database path (#4662)

c586d86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment