[PATCH] D99305: [docs] Document our norms around reverts

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 25 07:14:09 PDT 2021


probinson added a comment.

Please add words to the effect of:  When re-landing a reverted patch, the commit message should be updated to indicate the problem that was addressed (bot failure, cite the PR, whatever).

(Separately, my personal peeve is that reverting a revert leads to headlines with a chain of "Revert "Revert "Revert ..." messages, which is really less than helpful.  By adding this new section, I'll have a place to introduce a suggestion to set the message to "Reland "original message"" or some such.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99305



More information about the llvm-commits mailing list