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: remove node-gyp contributor workarounds / restrictions #30671

Draft
wants to merge 1 commit into
base: release/14.0.0
Choose a base branch
from

Conversation

MikeMcC399
Copy link
Contributor

@MikeMcC399 MikeMcC399 commented Nov 24, 2024

Additional details

This PR addresses documentation changes following the merge of PR #30633, which updated @electron/[email protected] to @electron/[email protected] in packages/server in the release/14.0.0 branch.

The update allows [email protected] to be used, with cascading restrictions relieved:

  1. The default Python 3.12 version of Ubuntu 24.* can be used.
  2. On Ubuntu 24.04 no downgrading to Python 3.11 necessary.
  3. Allows use of Ubuntu 24.10. (Downgrading Python on Ubuntu 24.10 is not available)
  4. On Windows no need to install a Windows 10 SDK additionally.

Building Cypress from source is then possible, without errors from node-gyp and without resorting to these workarounds.

Corresponding workaround instructions and restrictions are removed from CONTRIBUTING > Requirements.

Steps to test

Follow the modified instructions in CONTRIBUTING > Requirements for Cypress 14 beta, including the use of Node.js v20.18.0.

git clone https://github.com/cypress-io/cypress
cd cypress
git switch release/14.0.0
n auto # or set Node.js manually to 20.18.0
git clean -xfd
yarn

Confirm that the installation was successful and that the following log sequence shows no errors between the steps:

[5/6] Building fresh packages...
[6/6] Cleaning modules...

Remove any workarounds previously installed, which are no longer part of the updated CONTRIBUTING > Requirements instructions, as follows:

Ubuntu 24.04

If the system has been previously set up for Python 3.11, then execute the following before running yarn:

unset NODE_GYP_FORCE_PYTHON

Ubuntu 24.10

Python 3.11 is not available for Ubuntu 24.10, so no special preparatory steps are necessary.

Windows 11

In Visual Studio Installer remove any Windows 10 SDK installed. Install Python 3.12 through Microsoft Store.

Confirm that the error "could not find a version of Visual Studio 2017 or newer to use" no longer occurs.

How has the user experience changed?

This issue affects contributors only. Contributor instructions are changed for Ubuntu and Windows.

PR Tasks

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399
Copy link
Contributor Author

Submitted as draft and waiting for the release/14.0.0 branch to be rebased on develop which will then need some minor conflicts resolving in this PR.

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