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

[Documentation:Developer] Fix WSL issue installing Cypress #558

Merged
merged 3 commits into from
Oct 20, 2023

Conversation

RitaLei123
Copy link
Contributor

Changes: Added "If you are on WSL and run into an error, make sure you installed node.js on your desktop (Program Files) and not inside your local Submitty repository." in cypress.md (https://submitty.org/developer/testing/cypress)

WHY: I ran into a issue while installing cypress (I installed node.js in the wrong file) and this is to help others who might run into the same issue.

image

I ran into a issue while installing cypress and this is how others can prevent making the same mistake.
… issue installing Cypress

Changes: Added "If you are on WSL and run into an error, make sure you installed `node.js` on your desktop (Program Files) and not inside your local Submitty repository." in cypress.md (https://submitty.org/developer/testing/cypress)

WHY: I ran into a issue while installing cypress (I installed node.js in the wrong file) and this is to help others who might run into the same issue.
@RitaLei123 RitaLei123 changed the title [Documentation: Developer] Note for WSL users that may run into a issue installing Cypress [Documentation:Developer] Note for WSL users that may run into a issue installing Cypress Oct 15, 2023
@RitaLei123 RitaLei123 changed the title [Documentation:Developer] Note for WSL users that may run into a issue installing Cypress [Documentation:Developer] Note on fixing WSL issue installing Cypress Oct 15, 2023
@RitaLei123 RitaLei123 changed the title [Documentation:Developer] Note on fixing WSL issue installing Cypress [Documentation:Developer] Fix WSL issue installing Cypress Oct 15, 2023
Copy link
Member

@ziesski ziesski left a comment

Choose a reason for hiding this comment

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

Yes, looks good :)

@bmcutler bmcutler merged commit 9b6623e into main Oct 20, 2023
3 of 6 checks passed
@bmcutler bmcutler deleted the Cypress-Notes branch October 20, 2023 17:18
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.

3 participants