-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
✨(worker) Add support for pre
in properties
#4988
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 3bae03e:
|
👋 A preview of the new documentation is available at: http://664464159f02ef69ca801312--dubzzz-fast-check.netlify.app |
👋 A preview of the new documentation is available at: http://6644685934759e6c1c09df92--dubzzz-fast-check.netlify.app |
👋 A preview of the new documentation is available at: http://664653fa64a029033d29f973--dubzzz-fast-check.netlify.app |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4988 +/- ##
=======================================
Coverage 96.87% 96.87%
=======================================
Files 213 213
Lines 16296 16296
Branches 2667 2667
=======================================
Hits 15787 15787
Misses 509 509
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Description
The package
@fast-check/worker
was not compatible withfc.pre
. As such users were unable to rely on one of the core built-in primitives provided by fast-check. This PR adds support for it.Fixes #4473
Checklist — Don't delete this checklist and make sure you do the following before opening the PR
yarn bump
and flag the impacts properlyAdvanced
fc.pre
will behave correctly