[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: Gitlab Evaluation & Migration
From:       Boudewijn Rempt <boud () valdyas ! org>
Date:       2019-02-23 19:05:53
Message-ID: 3317270.u8ceFUVkNl () sebe
[Download RAW message or body]

On zaterdag 23 februari 2019 18:58:46 CET stefan@derkits.at wrote:

> "A lot" is probably a bit exaggerated, e.g. I don't really know where to 
> upload patches to Phabricator or create a pull request there, but do 
> understand how GitLab works.

I was talking about the Krita community, which uses Phabricator extensively in this \
way. I don't think you're familiar enough with the Krita community to make this \
comment. Also, not knowing some thing (how to find the Code Review link in the \
https://phabricator.kde.org/ homepage) while being familiar with another workflow \
doesn't mean that the first thing is hard, and the second one not.

> So I guess we have many different people in the community and many of 
> them can get used to change.

Everyone can get used to change; as long as the thing remains possible.

> > * clone the repo
> > * hack
> 
> * git commit
> * git push awesome-feature-branch

So, basically, what you're saying is that unless a person has push rights, they \
cannot collaborate? That's worse than I thought.

> * click on the link in the output
> 
> 
> > * add a bit of text explaining the change
> > * wait for me or dmitry to look at their patches
> 
> One more step for the first creation of a merge request. Not that much 
> different.
> 
> > They don't have push access to kde's git server at all, so I guess
> > 'git push my-fork HEAD' won't work in any case.
> 
> I guess this needs to change (with more fine grained permissions), the 
> whole Merge Request System is based on merging other branches to Master.
> Afaik uploading just a patch doesn't work in GitLab.

Well, that's too bad. Unless someone can explain to me how people can submit patches \
for review without having push rights, a migration seems impossible. It's already \
hard for some people to understand they need to create a KDE identity, but once \
they've got that, they should be able to offer patches for review.

-- 
https://www.valdyas.org | https://www.krita.org


[prev in list] [next in list] [prev in thread] [next in thread]