Skip to main content

[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

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



Back to the top