search

fastify / fastify-autoload

Require all plugins in a directory
MIT License
324 stars 66 forks source link

Merge `next` into `master` #396

Closed jsumners closed 2 months ago

jsumners commented 2 months ago

As titled.

jsumners commented 2 months ago

No idea what is wrong with these tests, and I have no time or inclination to figure out Jest.

jean-michelet commented 2 months ago

As far as I remember, it didn't always fail, but we should fix this. Have you tried to increase the default accepted timeout @jsumners?

FYI, I'm very busy with an onboarding for the next 2 weeks and have tons of things todo/learn. I should have more time after that, but I don't know Jest either so if you have any idea @climba03003

climba03003 commented 2 months ago

It is the problem I faced for local environment. I don't knows why it success in Github Actions before, and pop-up again now.

jean-michelet commented 2 months ago

Could this have been introduced by recent additions?

https://github.com/fastify/fastify-autoload/commits/next/

climba03003 commented 2 months ago

Could this have been introduced by recent additions?

It never run successfully in my local environment.