-
Notifications
You must be signed in to change notification settings - Fork 148
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
lookup: add eslint-plugin-jest #686
Conversation
This is mostly to have a code base using Jest. It would be preferable to include Jest itself instead. See #684
Codecov Report
@@ Coverage Diff @@
## master #686 +/- ##
=======================================
Coverage 94.41% 94.41%
=======================================
Files 26 26
Lines 823 823
=======================================
Hits 777 777
Misses 46 46 Continue to review full report at Codecov.
|
CI against current releases: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-pipeline/65/ |
Looks like the CI passed on all platforms besides AIX. |
@SimenB would you be so kind and skip this on AIX? It seems like yarn uses a functionality that does not exist on AIX before 7.1. |
Done! (I think) |
This is mostly to have a code base using Jest. It would be preferable to include Jest itself instead. See #684.
This is mostly to have a code base using Jest. It would be preferable to include Jest itself instead, but that is currently difficult due to some issues around colors.
See #684.
Checklist
here