[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