Skip to content

tools: [wip] use worker_threads for lint-js.js

Rodrigo Muino Tomonari requested to merge github/fork/Trott/lint-with-threads into master

Sometimes, you try something and it doesn't work out. I thought moving to worker_threads from cluster in lint-js.js would show at least a modest improvement. It seems about the same. Putting it up as a draft pull request to see if anyone has any additional ideas. Feel free to close if it seems stalled, or I'll get around to closing it.

The bottleneck seems to be CLIEngine#executeOnFiles() and I don't think there's much to do about that.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports