[PATCH] D89995: Make the post-commit review expectations more explicit with respect to revert

Sean Silva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 26 12:03:51 PDT 2020


silvas added a comment.

my 2 cents:

meta-point: this discussion should be happening on llvm-dev. it's clearly outside the scope of a patch review at this point.

good faith: we have to assume everybody in the community is acting on good faith. As Renato said, it just doesn't scale otherwise. If needed, we can develop a "bad actors" policy if bad actors show up in practice. In my experience, everyone is acting on good faith, but sometimes some 1-on-1 communication is needed for two parties to see each other's perspectives and come to consensus; back-and-forth on a patch review isn't necessarily the most productive.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89995



More information about the llvm-commits mailing list