Tipping the apple cart?
boud at valdyas.org
Tue Jul 2 07:42:26 BST 2019
On maandag 1 juli 2019 23:34:14 CEST Albert Astals Cid wrote:
> El dilluns, 1 de juliol de 2019, a les 9:42:34 CEST, Boudewijn Rempt va escriure:
> > Krita has switched from Phabricator to Gitlab a while ago, so maybe I can add our experience. It's not that great, though.
> > Bad:
> > * For new users who want to submit one or two patches, gitlab is way harder to use. They need much more help and handholding.
> Really? It uses the workflow all the other major review systems use, arc is a really weird tool (on some distros even hard to install)
Yes, really. I knew this comment was coming, but, yes, really.
> Or were you mostly getting patches sent as plain diffs uploaded to phabricator instead of by using arc?
Yes, nearly nobody uses arc.
> > * Gitlab has an exceedingly confusing UI where many options are very hard to find. The first thing I want to see when I get a MR is the diff,
> I guess that's your opinion, having the actual textual description and discussion is also very valuable, since you can get an overview on the MR quite fast.
> > and that means scrolling and hunting for a very small button.
> You mean the "Changes" button? I find it of adequate size
I guess that's _your_ opinion.
> > * gitlab is slow
> That's not the perception i have at all.
Try opening the changes tab for https://invent.kde.org/kde/krita/merge_requests/54. This makes my browser warn me two or three times that the page is using a lot of CPU and it takes ages before it succeeds.
> > * you cannot have more than one reviewer for a MR
> You can have as many reviewers as you want, i guess you mean you can't have more than one assignee.
Yes, and we need more than one assignee.
> > * using the label system for approving a MR is cumbersome
> What does this mean?
Check that MR I linked above and you can figure out yourself how we're using labels. It's the only mechanism we've found that would get close to the workflow needed to pass a MR from "needs review" to "needs changes" to "approved".
I can use gitlab, I will use gitlab, but it's not the huge newbie-enabler that it was touted to be. It sucks a bit, in practice.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: This is a digitally signed message part.
More information about the kde-core-devel