[PATCH] D148467: [clang-format] Add a new AfterCSharpProperty to BraceWrapping

Hunter T. via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 18 10:42:28 PDT 2023


StrangeRanger added a comment.

In D148467#4277960 <https://reviews.llvm.org/D148467#4277960>, @MyDeveloperDay wrote:

> I'm trying to decide if I put that option in this change or in a separate change (thoughts @HazardyKnusperkeks, @owenpan )

I'm not too privy to your workflow, but if by "in a separate change" you mean like "in a different PR", I'd probably say implement it in a different change, for the sake of revision history.


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

https://reviews.llvm.org/D148467



More information about the cfe-commits mailing list