<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks for alerting me. Why is it important to strip out this information? Also, might this be something we can change on the arcanist side (to not add to the commit message) rather than strip out on the git end?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 24, 2020 at 3:28 PM Fangrui Song 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">MaskRay added a comment.<br>
<br>
Hi, your git commit contains extra Phabricator tags. You can drop `Reviewers:` `Subscribers:` `Tags:` and the text `Summary:` from the git commit with the following script:<br>
<br>
  arcfilter () {<br>
          arc amend<br>
          git log -1 --pretty=%B | awk '/Reviewers:|Subscribers:/{p=1} /Reviewed By:|Differential Revision:/{p=0} !p && !/^Summary:$/ {sub(/^Summary: /,"");print}' | git commit --amend --date=now -F -<br>
  }<br>
<br>
`Reviewed By: ` is considered important by some people. Please keep the tag. (I have updated my script to use `--date=now` (setting author date to committer date))<br>
<br>
`<a href="https://reviews.llvm.org/D80978" rel="noreferrer" target="_blank">https://reviews.llvm.org/D80978`</a> contains a git pre-push hook to automate this.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D82225/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D82225/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D82225" rel="noreferrer" target="_blank">https://reviews.llvm.org/D82225</a><br>
<br>
<br>
<br>
</blockquote></div>