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(db push): make sure to stop migrate engine when it errors #8149
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.
Awesome!
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.
Nice, good catch!
Test not passing though yet.
If you can't reproduce this locally, let me know. Good catch @janpio, I got fooled by the grey cancel icon. I'll pay more attention next time 👍 |
Ok I'll merge main later and see what this is about |
That's the danger of the timeouts I think - they cancel the job, and do not fail it by default. |
Yes... but better that it times out after 10 minutes than 6 hours by default 😄 |
…#8149) * fix(db push): make sure to stop migrate engine when it errors Related prisma#7795 prisma#7990 * Restore package.json
Related
#7795
#7990
Thanks @millsp for finding the test that uncovered this 👍🏼