-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update readme #49
Update readme #49
Conversation
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.
@cadic Thanks for adding this. LGTM!!
README.md
Outdated
> Utilities library for WordPress E2E testing in the Cypress environment. | ||
|
||
## Prerequisites | ||
|
||
This library requires Cypress. Use https://github.com/10up/cypress-wp-setup to set up Cypress automatically. | ||
This library requires Cypress. Use [@10up/cypress-wp-setup](https://github.com/10up/cypress-wp-setup) to set up Cypress automatically. |
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.
@cadic because @10up/cypress-wp-setup
now includes this library, do you think we should note that behavior here?
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.
@cadic bumping this question for you^
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.
@dinhtungdu @jeffpaul added the comment about the current library included into the setup.
Description of the Change
Updated the readme:
run-all-cores.sh
Closes #47