[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG6c78711f106f: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy (authored by Flow, committed by njames93).

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

Flow added a comment. In D90972#2580020 , @njames93 wrote: > In D90972#2446448 , @Flow wrote: > >> In D90972#2394516 , @JonasToth >> wrote: >> >>> LGTM! thanks for fixing.

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

njames93 added a comment. In D90972#2446448 , @Flow wrote: > In D90972#2394516 , @JonasToth wrote: > >> LGTM! thanks for fixing. > > I am unable to commit this myself. The latest version of this change >

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

Flow requested review of this revision. Flow added a comment. In D90972#2394516 , @JonasToth wrote: > LGTM! thanks for fixing. I am unable to commit this myself. The latest version of this change

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

JonasToth accepted this revision. JonasToth added a comment. LGTM! thanks for fixing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90972/new/ https://reviews.llvm.org/D90972 ___ cfe-commits mailing

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

Flow updated this revision to Diff 303642. Flow added a comment. Use backticks in ReleaseNotes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90972/new/ https://reviews.llvm.org/D90972 Files: clang-tools-extra/clang-tidy/tool/CMakeLists.txt

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:76 +- The run-clang-tidy.py helper script is now installed in bin/ as + run-clang-tidy. It was previously installed in share/clang/. Please enclose run-clang-tidy.py,

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

sylvestre.ledru accepted this revision. sylvestre.ledru added a comment. This revision is now accepted and ready to land. Sounds great. You might want to ask feedback to other reviews too! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90972/new/

[PATCH] D90972: [clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy

Flow updated this revision to Diff 303638. Flow retitled this revision from "[clang-tidy] Install run-clang-tidy.py in bin/ not in share/clang/" to "[clang-tidy] Install run-clang-tidy.py in bin/ as run-clang-tidy". Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION