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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
