[PATCH] D147176: [clang-format] NFC ensure Style operator== remains sorted for ease of editing

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 30 11:56:08 PDT 2023


HazardyKnusperkeks added a comment.

In D147176#4232633 <https://reviews.llvm.org/D147176#4232633>, @MyDeveloperDay wrote:

> Honestly these developers are pretty much 'OGs' , I'm happy to tidy up after them if they don't quite follow our current conventions.  Obviously at any point you can add yourself to a review and add the appropriate tags.

Would you also please be so kind to sort the parsing function? I have currently no working clang-format development setup because of a change of hardware.

> Phabractor does support automatically assigned code reviewers via the "Owners" package, and via Herald rules. I could see if I can add one to at least auto assign the clang-format tag, that should bring it onto our radar
>
> H987 Clang Format,  Tags and Reviewers <https://reviews.llvm.org/H987>

That's nice, I've subscribed to the clang-format tag.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147176



More information about the cfe-commits mailing list