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.

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

docs: add usage of CYPRESS_DOWNLOAD_PATH_TEMPLATE in npmrc #4397

Closed
wants to merge 1 commit into from

Conversation

amoshydra
Copy link

@amoshydra amoshydra commented Mar 19, 2022

Add usage of CYPRESS_DOWNLOAD_PATH_TEMPLATE in npmrc.

This documentation PR should be merge once #19914 is addressed.
A PR addressing cypress-io/cypress#19914 is available at cypress-io/cypress#20698

Todo:

  • Mention that npm@7 and npm@8 is compatible with setting CYPRESS_DOWNLOAD_PATH_TEMPLATE in npmrc
  • Mention that npm@6 and yarn@1 is compatible with setting CYPRESS_DOWNLOAD_PATH_TEMPLATE in npmrc after cypress@x.y.z

@netlify
Copy link

netlify bot commented Mar 19, 2022

👷 Deploy request for cypress-docs pending review.
Visit the deploys page to approve it

🔨 Explore the source changes: 441516a

@CLAassistant
Copy link

CLAassistant commented Mar 19, 2022

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link

vercel bot commented Mar 19, 2022

@amoshydra is attempting to deploy a commit to the Cypress Team on Vercel.

A member of the Team first needs to authorize it.

When used in `.npmrc`, you will need to escape each dollar sign with a backslash

```ini
CYPRESS_DOWNLOAD_PATH_TEMPLATE=\${endpoint}/v${platform}-\${arch}/cypress.zip

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

${platform} was missed to be escaped

@rkrisztian
Copy link

Is this PR still needed? The original issue is closed.

@emilyrohrbough
Copy link
Member

Closing as stale. Please update with main if these changes are still relevant.

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.

None yet

4 participants