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

fix: add missing script for building wizard #15502

Merged
merged 3 commits into from
Mar 16, 2021

Conversation

elevatebart
Copy link
Contributor

As mentioned in #15450 we need to deliver the wizard again and probably update the Readme.md for @cypress/react.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Mar 15, 2021

Thanks for taking the time to open a PR!

dmtrKovalenko
dmtrKovalenko previously approved these changes Mar 15, 2021
```json
{
"experimentalComponentTesting": true,
"componentFolder": "src"
Copy link
Contributor

Choose a reason for hiding this comment

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

It's still required, isn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is not required anymore since we do not use preprocessors.
We should update the wizard, I will make a ticket.

Copy link
Contributor

Choose a reason for hiding this comment

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

No componentFolder is still required in order to find distinguish component specs and e2e specs.

Copy link
Contributor

Choose a reason for hiding this comment

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

Dmitriy is right.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why isn't it a default?
I'll fix it

@elevatebart
Copy link
Contributor Author

ensureNotAnimating is failing on windows for some reason. That is the error on appveyor.

Cypress throws the following error:

 animation_coords_history_invalid: 'coordsHistory must be at least 2 sets of coords',

I give up for tonight.

@jennifer-shehane jennifer-shehane merged commit 393a8ca into master Mar 16, 2021
elevatebart added a commit that referenced this pull request Mar 18, 2021
* fix: add missing script for building wizard

* docs: readme needs componentFolder

* fix: make build script work on windows
@elevatebart elevatebart deleted the fix/build-wizard-on-release branch March 19, 2021 00:47
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