-
-
Notifications
You must be signed in to change notification settings - Fork 949
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
docs: Faker and utility pages #1940
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #1940 +/- ##
==========================================
+ Coverage 99.61% 99.62% +0.01%
==========================================
Files 2561 2561
Lines 243239 243239
Branches 1274 1281 +7
==========================================
+ Hits 242301 242330 +29
+ Misses 913 884 -29
Partials 25 25 |
Should allLocales and allFakers be documented on these pages too? #1909 |
No, IMO they should be part of the localization page. |
I extracted the Faker jsdocs to its own PR #1960 to make reviewing them easier and let this PR focus on the infra/script side of things. |
120150c
to
18b5cad
Compare
I activated a deploy preview for this PR: |
@matthewmayer fixed |
This PR adds documentation pages for the
Faker
class and standalone utility methods.Netlify-Preview: https://deploy-preview-2--serene-sprite-f3ef50.netlify.app/