-
Notifications
You must be signed in to change notification settings - Fork 7
Update cypress dependency #32
Comments
Hey @yagudaev This change This Here is PR with my change: #33 After the fact. Once I released above change I realized that the What you did in package.json is expect. Programmer should be able to control what Cypress version he wants to use. You should explicitly specify Cypress version you want to use. {
"devDependencies": {
"@knapsack-pro/cypress": "3.1.0",
"cypress": "^4.11.0",
},
} Side thought. A while ago someone suggest to me to use |
I consulted this bug with @rafaltrzop who initially helped develop the project. It made me realized that I did not understand the full picture. @yagudaev You are using https://www.npmjs.com/package/npm-force-resolutions package to force installing a specific version of Cypress. This explains why in your project you have
If you wouldn't use This could lead to bugs because someone in the development environment may use new cypress features but on CI he will use To fix this problem we should use FYI @shadre |
Thank you @ArturT for the quick and detailed response. |
@yagudaev I've released a new version of I also added support for Cypress 5.x so |
Can we change the dependency of cypress to be
So that it works for 4.10, 4.11 and so on.
As a workaround I did:
The text was updated successfully, but these errors were encountered: