[lldb-dev] [cfe-dev] [RFC] Deprecate email code reviews in favor of Phabricator
Aaron Ballman via lldb-dev
lldb-dev at lists.llvm.org
Tue May 4 07:01:33 PDT 2021
On Tue, May 4, 2021 at 9:56 AM <paul.robinson at sony.com> wrote:
>
> > You're right that doing post-commit reviews on Phabricator is not
> > seamless---the rG link is not included anywhere. Hopefully that could be
> > fixed with some Phabricator configuration tweaks, like sending the commit
> > email to the -commits list.
>
> The commit email has a URL: link, e.g. this recent one (which has no
> Dnnnnn review):
>
> URL: https://github.com/llvm/llvm-project/commit/b04148f77713c92ee57b33b7b858ad18ce8d78e3
>
> Does that take you to a different place than the rG link would?
> Seems like they ought to go to the same place.
That sends you to the GitHub commit for the change. The corresponding
rG link would be:
https://reviews.llvm.org/rGb04148f77713c92ee57b33b7b858ad18ce8d78e3
~Aaron
More information about the lldb-dev
mailing list