[PATCH] D41956: [docs] Tweak update to Phabricator docs about setting repository for diffs uploaded via web

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 12 00:20:30 PST 2018


asb added a comment.

Yes, I'm sure the repository information is stored somewhere attached to the diff object - but it seems we don't use it. For this patch, I think the best way forwards is to advise users they can ignore the field when uploading the diff, and should enter it on the subsequent page. It's possible the workflow will be improved based on your upstream bug report, but we'd have to wait for the LLVM phabricator install to be updated anyway.

Thanks for all of your work on addressing the llvm-commits/cfe-commits subscription issue.


Repository:
  rL LLVM

https://reviews.llvm.org/D41956





More information about the llvm-commits mailing list