[all-commits] [llvm/llvm-project] 3c3714: Setup clang-format as an Arcanist linter

Siddhartha Bagaria via All-commits all-commits at lists.llvm.org
Mon Mar 30 12:03:05 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 3c371491a2d5023060391ff54b924bf1c7a0c41f
      https://github.com/llvm/llvm-project/commit/3c371491a2d5023060391ff54b924bf1c7a0c41f
  Author: Siddhartha Bagaria <sbagaria at grailbio.com>
  Date:   2020-03-30 (Mon, 30 Mar 2020)

  Changed paths:
    A .arclint
    A utils/arcanist/clang-format.sh

  Log Message:
  -----------
  Setup clang-format as an Arcanist linter

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




More information about the All-commits mailing list