This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] add -use-color option to clang-tidy-diff.py
ClosedPublic

Authored by ottojo on Feb 11 2021, 4:26 PM.

Details

Summary

add -use-color option to enable colored output in clang-tidy-diff.py

Clang-tidy seems to output color only when printing directly to terminal, but an option to force color-output has been added in https://reviews.llvm.org/D79477

Diff Detail

Event Timeline

ottojo requested review of this revision.Feb 11 2021, 4:26 PM
ottojo created this revision.
ottojo created this object with visibility "ottojo (Jonas Otto)".
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2021, 4:26 PM
ottojo changed the visibility from "ottojo (Jonas Otto)" to "Public (No Login Required)".Feb 11 2021, 4:27 PM
Eugene.Zelenko edited reviewers, added: hokein, aaron.ballman, njames93; removed: Restricted Project.Feb 11 2021, 5:22 PM
Eugene.Zelenko added a project: Restricted Project.
This revision is now accepted and ready to land.Feb 16 2021, 6:44 AM

I can't commit this myself, as i don't have commit access to the repo.

aaron.ballman closed this revision.Feb 18 2021, 6:15 AM

I can't commit this myself, as i don't have commit access to the repo.

Thanks for letting me know -- I've commit on your behalf in af06ff1cf87ecd387a65a6f7d4d00e0b06e983fb. Thank you for the patch!