aboutsummaryrefslogtreecommitdiff
path: root/utils
AgeCommit message (Collapse)AuthorFilesLines
2020-04-06Use in-tree clang-format-diff.py as Arcanist linterScott Linder1-13/+22
Summary: The only guarantee there seems to be in the clang-format packaging is that an executable called `clang-format` is in the PATH. Use the in-tree `clang-format-diff.py` to avoid assuming anything else. Also remove dead code for SVN repo and switch to `git diff-index` which is the git plumbing equivalent of `git diff` in this case. Reviewers: starsid, mehdi_amini, vitalybuka, fhahn, kadircet Subscribers: llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D77428
2020-04-03Revert "Use git-clang-format as Arcanist linter"Kadir Cetinkaya1-5/+5
This reverts commit 31b6e182f2ce957c86b961e21b9eb82bbcea7c59. As it was breaking `arc diff` workflow for multiple people.
2020-04-03Use --style file instead of --style LLVMVitaly Buka1-1/+1
LLVM includes .clang-format files which must be respected.
2020-04-03Use git-clang-format as Arcanist linterSiddhartha Bagaria1-5/+5
D49116 was using clang-format-diff because at the time of its writing, it needed to handle the subversion repo as well. Differential Revision: https://reviews.llvm.org/D77363
2020-03-30Setup clang-format as an Arcanist linterSiddhartha Bagaria1-0/+59
Summary: This uses clang-format-diff as a linter for Arcanist. `arc lint` flow, also run as part of `arc diff` unless skipped with `--nolint`, will now run the linter shell script on the changed files, and prompt the user to accept the suggested changes. Message when clang-format-diff is not installed: {F6654094} Example of the noise during code review when clang-format-diff is not installed: https://reviews.llvm.org/differential/changeset/?ref=1115809 Prompt when clang-format-diff is installed and suggests edits: {F6650223} Reviewers: probinson, scott.linder Reviewed By: scott.linder Subscribers: scott.linder, MyDeveloperDay, JonasToth, danilaml, JDevlieghere, dberris, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D49116