Update tap version to avoid intermittent test failures.

DavidTanner commented 6 months ago

Checklist

[x] run npm run test and npm run benchmark
[x] tests and/or benchmarks are included
[x] documentation is changed or added
[X] commit message and code follows the Developer's Certification of Origin and the Code of conduct

Even after #113 was closed, I can still get random errors in route.test.js. I have tested it over and over, and believe it is caused by the internals of Tap when collecting coverage. To test the theory I updated tap, and relevant tests, and can no longer repro the timeout.

I tested using a simple script that called npx tap repeatedly until a failure occurred.

gurgunday commented 6 months ago

Should there be a next branch?

The workflow change will stop testing for node v14

mcollina commented 6 months ago

I'm actually ok to ship a major. There aren't many issues with modules like this one.

mcollina commented 6 months ago

(to land as BlueOak is OSI-approved).

DavidTanner commented 5 months ago

@Uzlopak the license has been approved, can this PR be merged now?

fastify / fastify-swagger-ui

Update tap version to avoid intermittent test failures. #115

Checklist