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

Set video config to false by default #2718

Closed
sindresorhus opened this issue Nov 5, 2018 · 5 comments
Closed

Set video config to false by default #2718

sindresorhus opened this issue Nov 5, 2018 · 5 comments
Labels
topic: video 📹 type: breaking change Requires a new major release version type: unexpected behavior User expected result, but got another

Comments

@sindresorhus
Copy link

Current behavior:

The first time I run Cypress, it spams my project with lots of directories and files I don't need nor want. Merely running Cypress should not produce anything. I wasted a lot of time searching the docs for ways to disable this behavior.

Desired behavior:

Cypress should only produce these files if I explicitly ask for it. For example, npm init cypress or cypress init.

The video option should default to false. You should not assume people want this.

Cypress should not automatically scaffold stuff into cypress/fixtures, cypress/plugins, cypress/support. I could not find a way to disable this (which is very surprising), so I ended up forcing these directories to exist in Git so Cypress wouldn't do the scaffolding (touch cypress/plugins/.keep).

Steps to reproduce:

Can be reproduced by just running cypress run in a project.

Versions

Cypress 3.1.1

@konekoya
Copy link

konekoya commented Nov 5, 2018

Is this what you'd like to see? #619

@sindresorhus
Copy link
Author

Yes, and also changing the default of the video option.

@jennifer-shehane jennifer-shehane changed the title Don't spam my project with stuff Set video config to false by default Nov 5, 2018
@jennifer-shehane jennifer-shehane added type: feature New feature that does not currently exist stage: proposal 💡 No work has been done of this issue type: unexpected behavior User expected result, but got another and removed type: feature New feature that does not currently exist labels Nov 5, 2018
@brian-mann
Copy link
Member

Understand the concerns - it's probably time we changed this.

We can do a proposal for cypress init, and we've previously talked for awhile about updating the initial GUI scaffolding experience to be opt-in only.

@jennifer-shehane jennifer-shehane added the type: breaking change Requires a new major release version label Jun 25, 2020
@jennifer-shehane
Copy link
Member

Issue for cypress init to prevent scaffolding everything #619

@jennifer-shehane
Copy link
Member

Thanks for taking the time to explain the feature you'd like.

Our team has decided that adding the default for video config to false is not currently within our scope of feature work.

The cypress init work is something we intend to do and we have some team members working on improving the initial Cypress experience.

We will be closing this issue, but keeping it open for comments (which we do read). We will reconsider and reopen if we see this issue gains more 👍 or comments in support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: video 📹 type: breaking change Requires a new major release version type: unexpected behavior User expected result, but got another
Projects
None yet
Development

No branches or pull requests

4 participants