https://github.com/mydeveloperday requested changes to this pull request.
Yeah I'm kind of a not aligned on this approach of hijacking what feels like the wrong setting, it feels complicated after the change which probably means we are not thinking about it enough about what people might request in the future ```c++ // // License // <<-- added empty line <<-- added empty line (NewLinesAfterLicense: 2) #include <memory> #include <string> <<-- added empty line (NewLinesBetweenIncludeGroups: 2) <<-- added empty line #include <lib/memory> #include <lib/string> <<-- added empty line (NewLinesBetweenIncludeGroups: 2) <<-- added empty line #include <system/memory> #include <system/string> <<-- added empty line (NewLinesAfterIncludes: 3) <<-- added empty line <<-- added empty line namespace Bar { class Foo { }; ``` } // Bar https://github.com/llvm/llvm-project/pull/77918 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits