[PATCH] D152473: [clang-format] Add test case for issue 63170

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

In D152473#4409652 <https://reviews.llvm.org/D152473#4409652>, @phosek wrote:

> LGTM

@phosek can I ask you review https://llvm.org/docs/CodeReview.html

  When providing an unqualified LGTM (approval to commit), it is the responsibility of the reviewer to have reviewed all of the discussion and feedback from all reviewers ensuring that all feedback has been addressed and that all other reviewers will almost surely be satisfied with the patch being approved. If unsure, the reviewer should provide a qualified approval, (e.g., “LGTM, but please wait for @someone, @someone_else”). You may also do this if you are fairly certain that a particular community member will wish to review, even if that person hasn’t done so yet.

Whilst I understand you migth "like" the revert, I would ask that the when you do this, you say  `LGTM but please wait for the code owners".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152473