[libcxx-commits] [PATCH] D142180: [libc++] Enforce formatting for already formatted and new files and ignore the formatting of tests

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Jan 19 22:46:16 PST 2023


philnik created this revision.
philnik added reviewers: ldionne, Mordante, var-const, huixie90.
Herald added subscribers: abrachet, phosek, fedor.sergeev, arichardson.
Herald added a project: All.
philnik requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: libcxx-commits, sstefan1.
Herald added a project: libc++.
Herald added a reviewer: libc++.

It is quite confusing to newcomers that the formatting check gets mostly ignored. To fix that, enforce the formatting for new file and already formatted files, but ignore it for any files that aren't formatted already. We ignore the tests for now, since almost no test is formatted currently, and they are changed almost never.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D142180

Files:
  libcxx/utils/ci/buildkite-pipeline.yml
  libcxx/utils/ci/run-buildbot
  libcxx/utils/data/ignore_format.txt
  libcxx/utils/generate_ignore_format.sh

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D142180.490724.patch
Type: text/x-patch
Size: 36710 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230120/cfbc7221/attachment-0001.bin>


More information about the libcxx-commits mailing list