-
Notifications
You must be signed in to change notification settings - Fork 352
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
chore: fixes an issue with the e2e tests #4084
Conversation
π Benchmark resultsComparing with 1c7eb3b Package size: 355 MBβ¬οΈ 0.00% decrease vs. 1c7eb3b
Legend
|
Shouldn't the tests fail when there are unmet peer dependencies too? https://github.com/netlify/cli/runs/4866793722?check_suite_focus=true#step:6:376 |
It does not block the user (as it does not crash the install) Therefore the test should not fail. Only if something is blocking the user it should fail the test. It's basically a safeguard so that we know if we would break something for a user :) |
π Thanks for submitting a pull request! π
Summary
Fixes #4082
For us to review and ship your PR efficiently, please perform the following steps:
passes our tests.
A picture of a cute animal (not mandatory, but encouraged)