MarcusJohnson91 added a comment. In D75791#2044492 <https://reviews.llvm.org/D75791#2044492>, @MyDeveloperDay wrote:
> If you want me to land this for you, I'd feel more comfortable landing it if: > > a) We can land D80214: [clang-format] Set of unit test to begin to validate > that we don't change defaults <https://reviews.llvm.org/D80214> first > b) The Mozilla team have tested the impact (they clang-format their entire > code base I think) I'm ok with accepting commit access, and I agree lets get D80214: [clang-format] Set of unit test to begin to validate that we don't change defaults <https://reviews.llvm.org/D80214> in, and see if Mozilla, Microsoft, Google, etc has any comments; I'm just not sure of who to ping. Is there anything else that D80214: [clang-format] Set of unit test to begin to validate that we don't change defaults <https://reviews.llvm.org/D80214> needs? it looked pretty well fleshed out. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75791/new/ https://reviews.llvm.org/D75791 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
