Alex Bradbury via llvm-dev
2016-Sep-13 07:45 UTC
[llvm-dev] https://reviews.llvm.org/D23393
On 13 September 2016 at 07:59, Piotr Padlewski via llvm-dev <llvm-dev at lists.llvm.org> wrote:> This is exactly why we should switch to github. It is much easier for new > contributors to start with small fixes.There are two really quite separate issues here: 1) Patches, even trivial ones can sit unreviewed for long periods of time leading to a negative first time contribution experience. 2) Arguably, Phabricator provides a confusing interface for first-time or irregular users. There are efforts to improve this https://secure.phabricator.com/T5000 As for 1), this is something a few of us discussed at the LLVM Cauldron in Hebden Bridge last week which inspired me to give a lightning talk. Slides: <https://speakerdeck.com/asb/can-we-improve-the-experience-of-first-time-llvm-contributors> Video: <https://www.youtube.com/watch?v=32fmbEI9WrM> I'm intending to start an RFC thread when I get a chance. The tl;dr is I propose stealing Rust's highfive-bot and forming a team of eager volunteers who want to help improve response times for patches from first-time contributors. Best, Alex
David Chisnall via llvm-dev
2016-Sep-13 08:04 UTC
[llvm-dev] https://reviews.llvm.org/D23393
On 13 Sep 2016, at 08:45, Alex Bradbury via llvm-dev <llvm-dev at lists.llvm.org> wrote:> > 2) Arguably, Phabricator provides a confusing interface for first-time > or irregular users.I don’t think it’s fair to say that the Phabricator interface is confusing for first-time or irregular users. I use Phabricator on a couple of projects and review things with it quite regularly and I still find the interface obtuse. David -------------- next part -------------- A non-text attachment was scrubbed... Name: smime.p7s Type: application/pkcs7-signature Size: 3719 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20160913/679c7ade/attachment.bin>
Alfred Perlstein via llvm-dev
2016-Sep-13 08:17 UTC
[llvm-dev] https://reviews.llvm.org/D23393
On 9/13/16 1:04 AM, David Chisnall via llvm-dev wrote:> On 13 Sep 2016, at 08:45, Alex Bradbury via llvm-dev <llvm-dev at lists.llvm.org> wrote: >> 2) Arguably, Phabricator provides a confusing interface for first-time >> or irregular users. > I don’t think it’s fair to say that the Phabricator interface is confusing for first-time or irregular users. I use Phabricator on a couple of projects and review things with it quite regularly and I still find the interface obtuse. >+1, I'm a freebsd guy as well, we (freebsd) unfortunately use phabricator as well, it's just suboptimal.