njames93 added a comment.
In D80631#2065168 <https://reviews.llvm.org/D80631#2065168>, @MaskRay wrote:
> `arc` adds many unneeded tags from Phabricator. You can drop `Reviewers:`
> `Subscribers:` `Tags:` and the text `Summary:` with the following script:
>
> arcfilter () {
> arc amend
> 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 -
> }
>
>
> `Reviewed By: ` is considered important by some people
> (https://lists.llvm.org/pipermail/llvm-dev/2020-January/137889.html). You
> should keep the tag. (I started to use `--date=now` because some people find
> author date != committer date annoying. The committer date is usually what
> people care.))
Thanks for this, got it working nicely here
https://github.com/llvm/llvm-project/commit/5952125691571de9bd817551fb1baabe270e73f9
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80631/new/
https://reviews.llvm.org/D80631
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
