-
Notifications
You must be signed in to change notification settings - Fork 224
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
Add TESTING document and simplify README #3413
Add TESTING document and simplify README #3413
Conversation
move testing information from README to TESTING re-write testing information based on current test strategy, test specs. scripts and workflows
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.
Thank you very much for this PR. The new additions like npx cypress cache prune
are very much appreciated.
Tested every command, works.
What do you think, ready for merge?
Thanks for checking! Perhaps leave the PR for a day or two before merging to see if there are any other comments or points which are unclear? The Cypress cache description was previously in the README. I moved it to TESTING and shortened it. |
Sure. Additions may have been the wrong term - rewordings. I think it reads as good as new. |
Co-authored-by: Lars <wittmershaus.lars@gmail.com>
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.
Looks very good in general, thank 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.
LGTM!
This PR implements the suggestion from #3410.
Detailed testing information is moved from README: Testing into a new docs/TESTING.md file, which has re-worked contents that reflect the introduction of automatic testing. Some text from the original README which is no longer necessary has been removed.
Whilst this PR is open and not yet merged, the documents can be viewed under: