On Wed, 23 Sep 2020 at 16:05, Borislav Petkov <bp@xxxxxxxxx> wrote: > > Smita, > > pls sync the time of the box where you create the patch: > > Date: Fri, 4 Sep 2020 09:04:44 -0500 > > but your mail headers have: > > Received: from ... with mapi id 15.20.3370.019; Fri, 18 Sep 2020 14:49:12 +0000 > ^^^^^^^^^^^^^^^^^^ > > On Wed, Sep 23, 2020 at 07:07:17PM +0900, Punit Agrawal wrote: > > I know Boris asked you to add the reason for the Reported-by, but > > usually we don't track version differences in the committed patch. > > > > Boris, can you confirm if you want the Reported-by to be retained? > > How else would you explain what the Reported-by: tag is for on a patch > which adds a feature? > I think the question is why we are retaining this Reported-by header to begin with. Even though the early feedback is appreciated, crediting the bot for eternity for a version of the patch that never got merged seems a bit excessive. Also, it may suggest that the bot was involved in reporting an issue that the patch aims to fix but that is not the case. The last thing we want is Sasha's bot to jump on patches adding new functionality just because it has a reported-by line. So I suggest dropping the Reported-by credit as well as the [] context regarding v1