-
Notifications
You must be signed in to change notification settings - Fork 46
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
fix: migrate from jest to node test and c8 #1182
Conversation
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.
Please keep the test
inside the root test
folder.
It shouldn't move inside the src
.
b19ffb3
to
8c7b81b
Compare
You'll need to bump the minimum node.js version. You can do so by increasing the action version to v5 in fastify-passport/.github/workflows/ci.yml Line 20 in 53bb3b7
|
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.
Lint is failing
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
bd3361a
to
62bda9f
Compare
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
62bda9f
to
7bb7349
Compare
hey @climba03003 I updated based on your comments could you ptal? |
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Checklist
This PR adds borp and removes jest
Closes #1157
npm run test
andnpm run benchmark
and the Code of conduct