Skip to content
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.

Sign up for GitHub

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

Jump to bottom

perf(npm): only install cypress when needed #220

Merged
merged 1 commit into from
Aug 30, 2023
Merged

perf(npm): only install cypress when needed #220

merged 1 commit into from
Aug 30, 2023

Commits on Aug 30, 2023

  1. perf(npm): only install cypress when needed 

    We run `npm ci` in a number of places
and usually we do not need cypress to be installed.

In text a sample run of `npm ci` without cypress took 26 sec. instead of 46 sec.

This also prevents run failures due to network issues connecting to cypress.

Documentation for the environment variable used:
https://docs.cypress.io/guides/references/advanced-installation#Skipping-installation

Signed-off-by: Max <[email protected]>
    @max-nextcloud
    max-nextcloud committed Aug 30, 2023
    Configuration menu
    Copy the full SHA
    31b9209 View commit details
    Browse the repository at this point in the history