[PATCH] D129048: Rewording the "static_assert" to static assertion

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 21 08:02:35 PDT 2022


aaron.ballman added a comment.

In D129048#3668846 <https://reviews.llvm.org/D129048#3668846>, @ldionne wrote:

> In D129048#3668594 <https://reviews.llvm.org/D129048#3668594>, @philnik wrote:
>
>> Also, please wait for #libc <https://reviews.llvm.org/tag/libc/> approval next time.
>
> This, x1000.
>
> We go through the trouble of having excellent pre-commit testing and automatic review groups assigned to reviews, please don't bypass that.

We certainly will keep this in mind for the future, thanks for pointing it out! However, the precommit CI behavior confused multiple Clang contributors (I also thought the bot was misconfigured because Clang tests are never run against old Clang versions), the output did not clearly identify what was wrong or how to address it, and the difficulties with testing this locally are all things the libc++ maintainers should also keep in mind.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129048



More information about the cfe-commits mailing list