[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [babel-dev] please review patch on bug 221181, 470121 and 481655
|
Let's try that!
Thanks!
Kit Lo
Eclipse Babel Project Lead
IBM Eclipse SDK (IES) Globalization Technical Lead
IBM Rational solution for Collaborative Lifecycle Management (CLM) Globalization Technical Lead
Denis Roy ---11/11/2015 09:12:27 AM---Yes, that is Gerrit Code review. I suggest we review & merge your patches then do the switch.
From: Denis Roy <denis.roy@xxxxxxxxxxx>
To: babel-dev@xxxxxxxxxxx
Date: 11/11/2015 09:12 AM
Subject: Re: [babel-dev] please review patch on bug 221181, 470121 and 481655
Sent by: babel-dev-bounces@xxxxxxxxxxx
Yes, that is Gerrit Code review.
I suggest we review & merge your patches then do the switch.
I will walk us through the process.
Kit, do you agree or object?
Denis
On 11/10/2015 09:55 AM, Satoru Yoshida wrote:
> Hello, Denis.
>
> Does the Gerrit code review mean following web site ? :)
> example)
> https://git.eclipse.org/r/#/q/status:open
> https://git.eclipse.org/r/#/c/60031/
>
> Thank You
> Satoru Yoshida
>
> ----- Denis Roy <denis.roy@xxxxxxxxxxx>wrotes:
>>
>>
>> On a completely unrelated note, I think we should use Gerrit Code review. It is super easy and makes code review MUCH easier than what we are currently doing.
>> Kit, Satoru, do you want to try?
>>
>>
>> --Denis Roy @droy_eclipsehttp://eclipse.org/
>>
>> -------- Original message --------
>> From: Satoru Yoshida <raamat@xxxxxxxxxxxxxxxxx>
>> Date: 2015-11-09 4:53 PM (GMT-05:00)
>> To: Babel committers mailing list <babel-dev@xxxxxxxxxxx>
>> Subject: [babel-dev] please review patch on bug 221181, 470121 and 481655
>>
>> Hello, Denis and Kit.
>> I will feel happy if you review patch on bug 221181, 470121 and 481655.
>>
>> [221181] Search a specific string
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=221181
>>
>> [470121] (Reopen) scoreboard could be removed if no needed
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=470121
>>
>> [481655] jumpto.js and search.js do not exist.
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=481655
>>
>> I will make tag on pushed file in Git repository
>> if reviewed successfully. :)
>>
>> Thank You.
>> Satoru Yoshida
>> _______________________________________________
>> babel-dev mailing list
>> babel-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
>> https://dev.eclipse.org/mailman/listinfo/babel-dev
>
> _______________________________________________
> babel-dev mailing list
> babel-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/babel-dev
>
_______________________________________________
babel-dev mailing list
babel-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/babel-dev