-
-
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
chore: migrate jest-watcher to TypeScript #7843
Conversation
8d5e8de
to
459d0f8
Compare
prompt: string; | ||
}; | ||
|
||
export type AllowedConfigOptions = Partial< |
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.
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.
Huh, as I look a this, they're actually wrong. I took my list from the docs, but this is longer than that.
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.
Codecov Report
@@ Coverage Diff @@
## master #7843 +/- ##
==========================================
+ Coverage 64.36% 64.55% +0.19%
==========================================
Files 228 219 -9
Lines 8485 8370 -115
Branches 5 6 +1
==========================================
- Hits 5461 5403 -58
+ Misses 3022 2965 -57
Partials 2 2
Continue to review full report at Codecov.
|
7fd7945
to
cce57c9
Compare
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
I somehow missed that this depends on
jest-util
, so there's a@ts-ignore
in here 😅Built diff:
Test plan
Green CI