Skip to content

Conversation

@ilteoood
Copy link
Contributor

@ilteoood ilteoood commented Jan 6, 2025

Checklist

  • run npm run test and npm run benchmark

  • tests and/or benchmarks are included

  • documentation is changed or added

  • commit message and code follows the Developer's Certification of Origin
    and the Code of conduct

  • migrated to node test runner

  • removed semver dep, as not used anymore (the plugin always target node >= v20)

  • use fetch instead of http

@Fdawgs
Copy link
Member

Fdawgs commented Jan 6, 2025

Thanks @ilteoood, as noted in fastify/fastify-static#495, can you use c8 please?

@ilteoood
Copy link
Contributor Author

ilteoood commented Jan 6, 2025

Thanks @ilteoood, as noted in fastify/fastify-static#495, can you use c8 please?

Sure thing, done

@ilteoood ilteoood force-pushed the master branch 2 times, most recently from 02da776 to 927d5c4 Compare January 6, 2025 20:18
@ilteoood ilteoood requested a review from dancastillo January 7, 2025 18:09
@Fdawgs Fdawgs merged commit c746925 into fastify:master Jan 7, 2025
11 checks passed
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