[clang-tools-extra] [run-clang-tidy.py] Refactor, add progress indicator, add type hints (PR #89490)

Julian Schmidt via cfe-commits cfe-commits at lists.llvm.org
Sun May 26 16:57:14 PDT 2024


https://github.com/5chmidti commented:

Thanks for the cleanup, it looks overall good (w.r.t `asyncio`: I only know about `asyncio` what I read in this pr).

> Only print the filename after completion, not the entire Clang-Tidy invocation command. I find this neater but the behavior can easily be restored.

> By default print like in original script (command line), but add --progress switch that will change it to current behavior.

I'd prefer the original behavior as well, but printing only the filename can be added as a flag. Although, I'm not sure `--progress` is a good name for something that only changes if the command or file is printed, because the progress indicator should be emitted either way.

> Added an initial message with the number of files to process over the number of files in compilation database.

Maybe that was removed? Either way, I think it's +- not necessary, but I'm not against having it.

> More graceful shutdown would be welcome.

+1
IMO, that needs to be done before merging.


https://github.com/llvm/llvm-project/pull/89490


More information about the cfe-commits mailing list