-
Notifications
You must be signed in to change notification settings - Fork 29
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 a doc to introduce how to use e2e to test itself #119
Conversation
chunriyeqiongsaigao
commented
Sep 19, 2023
•
edited
edited
- update /test/e2e/concurrency/fail-fast/expected.yaml : use 'subtractor' in /test/e2e/concurrency/failfast/expected.yaml .
- add README.md to introduce how to use e2e-test (I will update this doc once I have finished writing it).
I think you missed the doc update? |
Thank you very much for the reminder. I will update this doc once I have finished writing it. |
@fgksgf Please take a look. |
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.
Please polish the doc. You can follow this template:
## Introduction
### File Structure
### How it works
## How to add new cases
(use an example)
## Tips
- how to run e2e tests locally
- you may need to split your PR to pass e2e tests in CI
- ...
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, thanks for your contribution!