<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 5, 2021 at 10:47 AM Philip Reames via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">reames added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/docs/DeveloperPolicy.rst:330<br>
+* In general, if the author themselves would revert the change per these<br>
+  guidelines, we encourage other contributors to do so as a favor to the<br>
+  author.  This is one of the major cases where our norms differ from others;<br>
----------------<br>
probinson wrote:<br>
> reames wrote:<br>
> > echristo wrote:<br>
> > > reames wrote:<br>
> > > > echristo wrote:<br>
> > > > > I think wording wise we can just remove the "... as a favor to the author." here and reword a little below...<br>
> > > > I left this one as is.  I took all your other edits, but taking this out seems to loose an important point to me.  <br>
> > > I followed up offline here with an article that would help illustrate my point, but for me not having the "I'm doing you a favor" is important. It's not a favor, it's just how we work. There's no obligation or reciprocity expected which the text gives the impression of.<br>
> > You did, my apologies for not acknowledging that.<br>
> > <br>
> > I read the article you sent, it was definitely interesting, but it didn't change my take on this.<br>
> > <br>
> > I'm going to defer to Chris on this.  If he wants a change, I'll make it.  If he's okay with the current wording, I'll leave it as is.<br>
> Maybe "courtesy" rather than "favor" would have less implication of an obligation?  Overall we are expected to be courteous to one another.<br>
Great suggestion, that would work for me.  Eric?<br></blockquote><div><br></div><div>Could work for now. Could you reframe the wording around "we are being courteous to each other by helping keep top of tree working well"? I'd support wording in those terms - it doesn't phrase things by imposing obligations onto others.</div><div><br></div><div>-eric</div></div></div>