[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hiding /web/install
From: |
David Kastrup |
Subject: |
Re: [PATCH] hiding /web/install |
Date: |
Thu, 22 Dec 2011 12:54:23 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.92 (gnu/linux) |
James <address@hidden> writes:
> Alberto
>
> 2011/12/22 Alberto Simões <address@hidden>:
>> Hey, James
>>
>>
>> On 12/22/11 11:42 , James wrote:
>>>
>>> Do you need me to upload the patch for review to Rietveld for you?
>>
>>
>> Forgot about it.
>> What is Rietveld? :$
>>
>> Probably if you point me to it I can do it.
>> Thanks.
>
> http://codereview.appspot.com/
>
> You need a gmail account.
Not really. You need to create an _account_ at Google, but that does
not imply Gmail at all. And it is far too onerous to create reviews
(and announcements of them) manually there.
Instead use
git cl upload
for it. I think the details should be in the Contributor's Guide.
--
David Kastrup
- Re: [PATCH] hiding /web/install, James, 2011/12/22
- Re: [PATCH] hiding /web/install, James, 2011/12/22
- Re: [PATCH] hiding /web/install, Alberto Simões, 2011/12/22
- Re: [PATCH] hiding /web/install, James, 2011/12/22
- Re: [PATCH] hiding /web/install, Alberto Simões, 2011/12/22
- Re: [PATCH] hiding /web/install, James, 2011/12/22
- Re: [PATCH] hiding /web/install, Alberto Simões, 2011/12/22
- Re: [PATCH] hiding /web/install,
David Kastrup <=
- Re: [PATCH] hiding /web/install, Graham Percival, 2011/12/22