[PATCH] D49116: Setup clang-format as an Arcanist linter

MyDeveloperDay via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 25 06:41:12 PST 2019


MyDeveloperDay added a comment.

Slight aside there is already a large amount of things in LLVM that don't work in the cmd.exe prompt (or even sometimes in cygwin), so I personally wouldn't set that as a gate for this.

However once you add a `.arclint` file in the root then I believe all  `arc diff` will run the lint UNLESS you do  `arc diff --no-lint`, but that's probably been mentioned before.

That could easily catch people out.. but there are workarounds using arc alias (https://secure.phabricator.com/Q113)

@JonasToth we could have a got at making an .arclint configuration that would run the validate_check.py with the --rst option to pass a rst filename


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D49116/new/

https://reviews.llvm.org/D49116





More information about the llvm-commits mailing list