Skip to content

Conversation

@unikitty37
Copy link
Contributor

If the user already has a globally-installed Cypress, they may not want one created in their app's node_modules folder (particularly if the Rails app is an API-only backend!)

As passing any value to install_cypress_with that isn't yarn or npm skips installation, I've chosen skip here. As long as nobody releases a package manager called skip this should be fine 😁

If the user already has a globally-installed Cypress, they may not want one created in their app's `node_modules` folder (particularly if the Rails app is an API-only backend!)

As passing any value to `install_cypress_with` that isn't `yarn` or `npm` skips installation, I've chosen `skip` here. As long as nobody releases a package manager called `skip` this should be fine 😁
@grantspeelman
Copy link
Collaborator

@unikitty37 I like it 👍

@grantspeelman grantspeelman merged commit 7f97fc7 into shakacode:master Nov 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants