Skip to content

Conversation

@dancastillo
Copy link
Member

Checklist

This PR migrates from tap to node:test and c8

@Uzlopak
Copy link
Contributor

Uzlopak commented Aug 18, 2024

@dancastillo

Can you please have a look at your previous migrations and check if c8 is using the --100 flag everywhere?

Copy link
Member

@gurgunday gurgunday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Uzlopak Uzlopak changed the title fix: migrate from tap to node:test and c8 chore: migrate from tap to node:test and c8 Aug 19, 2024
Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Uzlopak Uzlopak merged commit 36bd0e9 into fastify:master Aug 19, 2024
@dancastillo dancastillo deleted the migrate-node-test-c8 branch August 21, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants