-
Notifications
You must be signed in to change notification settings - Fork 640
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(expect): minor doc tweaks #5501
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5501 +/- ##
=======================================
Coverage 96.30% 96.30%
=======================================
Files 463 463
Lines 37669 37669
Branches 5561 5561
=======================================
Hits 36276 36276
Misses 1351 1351
Partials 42 42 ☔ View full report in Codecov by Sentry. |
@@ -19,14 +17,16 @@ | |||
* expect(mockFn).toHaveBeenCalledTimes(1); | |||
* }); | |||
* ``` | |||
* | |||
* @module |
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.
Fixes module docs not being displayed.
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.
Good catch
* const x = 6 * 7; | ||
* expect(x).toEqual(42); | ||
* expect(x).not.toEqual(0); | ||
* |
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.
Aligns with other packages which usually have an example first.
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
No description provided.