[PATCH] D78867: [docs] avoid 'arc land' command
Sergei Trofimovich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 28 11:17:33 PDT 2020
trofi marked 2 inline comments as done.
trofi added inline comments.
================
Comment at: llvm/docs/Phabricator.rst:224
- arc land D<Revision>
+ arc land arcpatch-D<Revision>
----------------
trofi wrote:
> mehdi_amini wrote:
> > I didn't know we were documenting `arc land`, I think we should discourage it heavily instead.
> >
> > This is creating commit message with a lot of cruft inside: all the Phabricator tags should be removed before pushing IMO.
> >
> Oh, TIL.
>
> I can reword the section to avoid `arc land`. Would you be fine to review it or maybe you can suggest someone else as well?
Reworded to discourage `arc land`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78867/new/
https://reviews.llvm.org/D78867
More information about the llvm-commits
mailing list