checkupjs / checkup

A health checkup for your project.
https://checkupjs.github.io/
MIT License
77 stars 23 forks source link

[feature] be able to run multiple tasks in separate threads #527

Open gabrielcsapo opened 4 years ago

gabrielcsapo commented 4 years ago

Motivation

Looking at the runTasks function, there is some room for improvement on systems that have multiple cores. Tasks could be threaded off using something like https://www.npmjs.com/package/piscina.

https://github.com/checkupjs/checkup/blob/a11f17c22ada885f137dbd18d05ae21410ed922f/packages/cli/src/task-list.ts#L97-L113

scalvert commented 4 years ago

I'd prefer not to jump to using threads unless absolutely needed. Currently, the eachTasks function uses p-map, and unless there's a super compelling reason to switch, I'd prefer to leave this as is.

What problem are you trying to solve?