On zaterdag 23 februari 2019 18:58:46 CET ste...@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
