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
Add test #411
Comments
The only code here that has some logic that could be tested and uses external dependencies is now the publish script and not the package itself, as far as I can tell. |
(Corollary: whichever contrived test we may add to https://github.com/prisma/engines-wrapper/tree/main/packages/engines-version, it probably won't help us much be able to merge Renovate PRs with confidence since it's not the code that will likely break if something is wrong with the updated dependencies) |
Indeed, we could have some "dry run mode" for the publish script and test what that does. |
Since #441 we have a build action which will let us know if TS compilation fails. |
I made it a dry publish in #442 |
I think we can close this now |
https://github.com/prisma/engines-wrapper/tree/main/packages/engines-version doesn't have any test
It would be better to have some test, so we can safely merge Renovate PRs like #240
The text was updated successfully, but these errors were encountered: