-
Notifications
You must be signed in to change notification settings - Fork 3.4k
docs: add contributor quick start guide for faster onboarding #32907
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
base: develop
Are you sure you want to change the base?
docs: add contributor quick start guide for faster onboarding #32907
Conversation
- Add 15-minute onboarding guide for new contributors - Include essential setup, workflows, and high-impact areas - Provide copy-paste commands and debugging tips - Complement existing CONTRIBUTING.md with focused approach This reduces onboarding friction and provides a clear path to first contribution.
|
|
|
| # Install dependencies (this takes a few minutes) | ||
| yarn install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In order to use Yarn, it needs to be installed. There are however no instructions in the Quick Setup to install Yarn.
| # Check your Node.js version (required: ^20.1.0 || ^22.0.0 || >=24.0.0) | ||
| node --version | ||
|
|
||
| # If wrong version, use nvm | ||
| nvm install 22.0.0 | ||
| nvm use 22.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Attempting to run yarn install with Node.js 22.0.0 (assuming that Yarn v1 Classic is already installed) results in the error message:
$ yarn install
yarn install v1.22.22
[1/6] Validating package.json...
error [email protected]: The engine "node" is incompatible with this module. Expected version ">=22.19.0". Got "22.0.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
| # If wrong version, use nvm | ||
| nvm install 22.0.0 | ||
| nvm use 22.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm is not installed by default. It is one of several optional Node.js version managers. Instructions to install it are missing. There are different versions of nvm, depending on the operating system (Linux, macOS or Windows).
| ### 1. 🐛 Quick Bug Fixes | ||
| **Where**: `packages/driver/src/` - Core test execution logic | ||
| **Why**: Direct user impact, well-defined problems | ||
| **Start here**: Issues labeled `good first issue` + `driver` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no label driver
See https://github.com/cypress-io/cypress/labels
| ### 2. 🔧 CLI Improvements | ||
| **Where**: `cli/lib/` - Command-line interface | ||
| **Why**: Affects every Cypress user, easier to test locally | ||
| **Start here**: Issues labeled `cli` + `enhancement` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no label enhancement
See https://github.com/cypress-io/cypress/labels
| ### 3. 📚 Component Testing | ||
| **Where**: `npm/react/`, `npm/vue/`, `npm/vite-dev-server/` | ||
| **Why**: Growing area, lots of room for improvement | ||
| **Start here**: Issues labeled `component testing` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no label component testing. There is however a label CT.
See https://github.com/cypress-io/cypress/labels
| yarn install | ||
|
|
||
| # Verify everything works | ||
| yarn workspace @packages/driver test:unit:ci --run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command fails:
$ yarn workspace @packages/driver test:unit:ci --run
yarn workspace v1.22.22
yarn run v1.22.22
error Command "test:unit:ci" not found.
| ### Run Specific Package Tests | ||
| ```bash | ||
| # Driver (core test execution) | ||
| yarn workspace @packages/driver test:unit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command fails:
$ yarn workspace @packages/driver test:unit
yarn workspace v1.22.22
yarn run v1.22.22
error Command "test:unit" not found.
| cd cli && yarn test | ||
|
|
||
| # Server (backend logic) | ||
| yarn workspace @packages/server test:unit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command fails to start:
$ yarn workspace @packages/server test:unit
yarn workspace v1.22.22
yarn run v1.22.22
error Command "test:unit" not found. Did you mean "test-unit"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR appears to contain untested AI-generated content. Regardless of how it was generated, it contains multiple errors. In its present form, the document is not usable.
The Cypress.io team should evaluate whether the creation of an additional document on top of the https://github.com/cypress-io/cypress/blob/develop/CONTRIBUTING.md reference document is desired and needed. It would mean a duplication of instructions contained in the CONTRIBUTING and individual README documents.
@lucgonp has also not signed the Contributor License Agreement, meaning that the PR is not eligible for acceptance.
I did not review the whole document. After commenting on multiple incorrect instructions / missing information at the beginning of the document, I then stopped.
This reduces onboarding friction and provides a clear path to first contribution.
Additional details
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation?type definitions?Note
Adds
CONTRIBUTING.quickstart.md, a concise 15-minute onboarding guide with setup, workflows, commands, debugging tips, and key resources.CONTRIBUTING.quickstart.mdquick start guide covering:Written by Cursor Bugbot for commit 4e14300. This will update automatically on new commits. Configure here.