-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
updated docs regarding testSequencer #9174
updated docs regarding testSequencer #9174
Conversation
Hi saiumesh535! Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks! |
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.
Thanks! Could you updated the versioned docs as well? 24.8 and 24.9 have it
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
1 similar comment
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
Codecov Report
@@ Coverage Diff @@
## master #9174 +/- ##
=======================================
Coverage 64.82% 64.82%
=======================================
Files 279 279
Lines 11740 11740
Branches 2885 2885
=======================================
Hits 7610 7610
Misses 3511 3511
Partials 619 619 Continue to review full report at Codecov.
|
Sure, let me update it. Thanks! |
@saiumesh535 ping 🙂 |
@SimenB |
No need to apologize, we're all doing this in our spare time! 🙂 |
9a1b92e
to
1c391ea
Compare
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.
Updated versioned docs. Skipped 25.1 and 25.3 as they'll soon be folded into 25.x with #9956
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
While trying to make run jest test cases in the required sequence, it was a little difficult to find and how to add testSequencer config. So Added example config for testSequencer. It will help and makes easier for users to run.
Test plan
Not required as we updated only docs.