Skip to content
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

fix: reset database for sqlite and windows #747

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

nikophil
Copy link
Member

@nikophil nikophil commented Dec 14, 2024

fixes #746

I found the fix thanks to laravel/framework#21720

At start, I wanted to add a unique CI permutation with windows, but it is incompatible with this, we're getting an error "Container operations are only supported on Linux runners".

The only solution would be to add yet another job, which I'm reluctant to do. But the tests actually passed with a window runner. Let's say we'll add this new job if a new bug with windows occurs

@nikophil nikophil force-pushed the test/add-windows-permutation branch 19 times, most recently from 3db8b8a to 4b67273 Compare December 14, 2024 14:01
@nikophil nikophil force-pushed the test/add-windows-permutation branch from 4b67273 to bab835a Compare December 14, 2024 14:02
@nikophil nikophil changed the title tests: add CI permutation with windows fix: reset database for sqlite and windows Dec 14, 2024
@nikophil nikophil marked this pull request as ready for review December 14, 2024 14:03
@nikophil nikophil merged commit f4ba5d8 into zenstruck:2.x Dec 14, 2024
54 checks passed
@nikophil nikophil deleted the test/add-windows-permutation branch December 14, 2024 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Error Resetting SQLite Database Introduced in 2.3.1
1 participant