Using GitHub PRs?

248 views
Skip to first unread message

Brad Fitzpatrick

unread,
Jun 4, 2019, 11:07:30 AM6/4/19
to per...@googlegroups.com
Perkeepers,

We're thinking of moving off Gerrit and simplifying the contribution experience for most users.

GitHub's built-in code reviews aren't quite as powerful as Gerrit yet, but there are 26 other code review tools that integrate well with GitHub (of which Gerrit is not one of them) that we could use later if we wanted. When we first started using Gerrit, GitHub PRs were much worse. Nowadays I don't think Gerrit's better enough to be worth the complexity of having two systems that don't work well together and we force people to learn.

Any opinions? Anybody object if we drop Gerrit?

- Brad

Mark Turner

unread,
Jun 4, 2019, 12:38:00 PM6/4/19
to per...@googlegroups.com
Coming from someone who has failed to contribute at all, I think anything you can do to lower the barrier of entry is great. But I’m just shouting from the peanut gallery so take my opinion with a grain of salt.

Zoran Zaric

unread,
Jun 4, 2019, 12:38:15 PM6/4/19
to per...@googlegroups.com


> Am 04.06.2019 um 17:07 schrieb Brad Fitzpatrick <br...@danga.com>:
>
> Perkeepers,
>
> We're thinking of moving off Gerrit and simplifying the contribution experience for most users.

Absoute +1 from me as somebody who wanted to contribute something simple but was put off because of the overwhelming Gerrit-process.

Thanks,
Zoran

clive boulton

unread,
Jun 4, 2019, 12:48:24 PM6/4/19
to per...@googlegroups.com
I too have potential commits sitting in a perkeep git fork for Android client and build system because duplicating CLs to Gerrit took too much duplication time.    

--
You received this message because you are subscribed to the Google Groups "Perkeep" group.
To unsubscribe from this group and stop receiving emails from it, send an email to perkeep+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/perkeep/3A51B827-3B26-4553-BE60-C5CEE05D1844%40zoranzaric.de.
For more options, visit https://groups.google.com/d/optout.

Paul Lindner

unread,
Jun 4, 2019, 12:53:36 PM6/4/19
to per...@googlegroups.com
+1  -- too much activation energy makes it hard for me to get back in the game :(

Lance Wicks

unread,
Jun 4, 2019, 4:54:55 PM6/4/19
to per...@googlegroups.com
As much as I love gerritt, it is a bit of a barrier to overcome to contribute.
GitHub is defacto standard, which pains me as it's very much a centralisation. But I'd not object.

--
You received this message because you are subscribed to the Google Groups "Perkeep" group.
To unsubscribe from this group and stop receiving emails from it, send an email to perkeep+u...@googlegroups.com.

Saulius Gurklys

unread,
Jun 4, 2019, 5:07:15 PM6/4/19
to per...@googlegroups.com
I think simple proces is always better, of cource if  quality of code is ensured.
+1.

--
S4uliu5 

Ian Denhardt

unread,
Jun 4, 2019, 5:13:33 PM6/4/19
to Lance Wicks, per...@googlegroups.com
Quoting Lance Wicks (2019-06-04 16:54:41)

> GitHub is defacto standard, which pains me as it's very much a
> centralisation. But I'd not object.

I'm pretty much in the same spot.

That said, I suspect most of the barrier is that the use of the tool is
quite different; change ids, having to set up tooling locally... Folks
are used to the fork & pr workflow. My gut is that moving to some other
platform (either hosted or on our own infrastructure) with a
sufficiently similar workflow would drop *most* of the barrier --
GitLab, Gogs/Gitea , etc. are all similar, and generally you can even
log in with your GitHub account.

But it probably makes more sense to just use GitHub, if for no other
reason than the bug tracker is there. I guess part of why I don't have a
strong objection despite centralization is I don't really feel like
we're locked in to the provider; if GitHub starts doing things we find
unacceptable, I don't think it will be too onerous to migrate to a
competitor or our own infrastructure. If that starts to change I will
start to worry.

-Ian

Will Norris

unread,
Jun 5, 2019, 10:50:53 AM6/5/19
to per...@googlegroups.com
nothing much add beyond what others have said.  just that yeah, this probably makes sense at this point.

Brad Fitzpatrick

unread,
Jun 5, 2019, 12:58:06 PM6/5/19
to per...@googlegroups.com
All seem in favor, so it's now done:

GitHub is now the upstream. Please update your devcam binary & git config origin.

We can finish up any pending reviews on Gerrit and we'll manually cherry-pick and push them to GitHub as needed. (The automatic Gerrit to GitHub sync is now off.)


On Tue, Jun 4, 2019 at 8:07 AM Brad Fitzpatrick <br...@danga.com> wrote:
Reply all
Reply to author
Forward
0 new messages