search

fastify / fastify-vite

Fastify plugin for Vite integration.
MIT License
861 stars 72 forks source link

convert for loops to Promise.all of array.map #70

Closed davidmeirlevy closed 2 years ago

davidmeirlevy commented 2 years ago

trying to improve the bootstrap a bit with parallel write files.

davidmeirlevy commented 2 years ago

@mcollina can you elaborate? I would love to know what.

galvez commented 2 years ago

@davidmeirlevy I think Matteo means it's hard to handle errors this way. We can still try and parallelize with Promise.all(), I think that was actually my intention — but we need to refactor the code to handle errors and maybe add a resume mechanism when there is one.

mcollina commented 2 years ago

Thanks for translating Jonas. Error handling in that case would be extremely hard.