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

Deprecation updates for the nodeVersion config option #4194

Merged
merged 4 commits into from
Nov 4, 2021

Conversation

mjhenkes
Copy link
Member

@mjhenkes mjhenkes commented Nov 2, 2021

As a part of issue #18684, This PR updates the documentation to declare the nodeVersion config as deprecated as well as explain the default system and behaviour, to use the same node binary that was used to launch cypress from the cli.

@mjhenkes mjhenkes self-assigned this Nov 2, 2021
@CLAassistant
Copy link

CLAassistant commented Nov 2, 2021

CLA assistant check
All committers have signed the CLA.

Comment on lines 144 to 146
As of version 9.0.0, Cypress will use the same version of node that was used to launch the cypress App via the CLI.

Prior to version 9.0.0, Cypress would use the version of node bundled with Cypress itself.
Copy link
Member

Choose a reason for hiding this comment

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

We don't keep notes like this in our docs for previous versions, the docs always reflect current version. In some cases we have a 'History' section in the file to explain changes

Can you update this section to reflect the truth of the new version?

content/guides/references/configuration.md Outdated Show resolved Hide resolved
content/guides/references/configuration.md Outdated Show resolved Hide resolved
mjhenkes and others added 2 commits November 3, 2021 06:26
Co-authored-by: Jennifer Shehane <jennifer@cypress.io>
@brian-mann brian-mann self-requested a review November 3, 2021 17:49
Copy link
Member

@jennifer-shehane jennifer-shehane left a comment

Choose a reason for hiding this comment

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

Just reread through the 'writing a plugins' page and it doesn't make sense to have this alert anymore I think, so I just removed the whole section. Looks good, we'll merge this into 9.0 as soon as the code PR is merged.

@mjhenkes mjhenkes merged commit f31cf85 into 9.0-release Nov 4, 2021
@mjhenkes mjhenkes deleted the nodeVersion branch November 4, 2021 18:46
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