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

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Feb 2 08:11:12 PST 2023


ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

LGTM, but let's also update the documentation at https://libcxx.llvm.org/Contributing.html to mention that new files need to be run through clang-format.



================
Comment at: libcxx/utils/ci/run-buildbot:218
+        echo "It looks like the list of not formatted files has changed."
+        echo "If a file is now formatted, remove the file name from "
+        echo "libcxx/utils/data/ignore_format.txt. Otherwise you have to fix the"
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142180



More information about the libcxx-commits mailing list