ZedThree / clang-tidy-review

Create a pull request review based on clang-tidy warnings
MIT License
90 stars 45 forks source link

Use task queue to spawn multiple processes of tidy #126

Closed bwrsandman closed 2 months ago

bwrsandman commented 5 months ago

This PR streamlines build_clang_tidy_warnings to be easily dispatched by a multi-process queue. Each process stores the fixes in a temporary directory. The queue will dispatch a clang-tidy process per file to check. This way more complex files can run at the same time as less complex ones. Added a function to combine fixes after the queue is done running.

The design is copied from llvm's run-clang-tidy.py script and adapted for the current code.

Things I've noticed running this:

Depends on:

Closes #124

bwrsandman commented 3 months ago

@ZedThree fixes done

bwrsandman commented 3 months ago

@ZedThree ping!