[Development] [Qt-creator] gerrit-speak
Konstantin Tokarev
annulen at yandex.ru
Fri Dec 14 15:35:55 CET 2012
14.12.2012, 15:02, "Sergio Ahumada" <sergio.ahumada at digia.com>:
> On 12/14/2012 11:54 AM, Bache-Wiig Jens wrote:
>
>>> Oh well, if you already feel offended by this phrasing, I guess you
>>> should get a thicker skin ... we've people from very different
>>> cultures and with varying English language skills in the community, so
>>> you should just take things with a pinch of salt in general.
>>>
>>> Anyway, the sentence is from the upstream gerrit project, and it
>>> actually describes pretty good what it means, technically:
>>>
>>> "you shouldn't submit this" - I don't think this is ready to go in as
>>> it is.
>>> "I would prefer" - ..but at the same time this isn't a veto, so if
>>> another approver thinks differently (with reasons), please go ahead.
>> Actually looking at the android version of gerrit which I presume would
>> be using the upstream version they have omitted the comments entirely,
>> merely using a "Code-Review +1" comment.
>> They also have a helpful "browse projects" link in the header and their
>> CSS looks cleaner. Perhaps we are simply not up to date or we actually
>> applied those comments locally?
>>
>> See https://android-review.googlesource.com/#/q/status:open,n,z
>>
>> Jens
>
> That's 2.5 version and we based ours in 2.2.1
>
> Some attempts to go for 2.2.2 have been made without much success
> https://bugreports.qt-project.org/browse/QTQAINFRA-466
>
> They just change the implementation between minor releases.
> Also, it seems like they don't want our features to be up-streamed, see
> https://groups.google.com/forum/?fromgroups=#!topic/repo-discuss/gXjbuhfW0tg
> for the topic feature discussion, so it' even harder to be in sync.
Well, Gerrit 2.4 introduced user-defined submit rules, and Gerrit 2.5 introduced plugins.
I think it should be possible to implement everything needed on top of these features now.
--
Regards,
Konstantin
More information about the Development
mailing list