Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(359)

Issue 10636014: Add rlz/OWNERS, remove rlz from .gitignore (Closed)

Created:
8 years, 6 months ago by Nico
Modified:
8 years, 6 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add rlz/OWNERS, remove rlz from .gitignore BUG=none TEST=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=143698

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -4 lines) Patch
M .gitignore View 1 chunk +0 lines, -1 line 0 comments Download
A + rlz/OWNERS View 1 chunk +2 lines, -3 lines 2 comments Download

Messages

Total messages: 5 (0 generated)
Nico
8 years, 6 months ago (2012-06-22 19:44:14 UTC) #1
Glenn Wilson
lgtm https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS File rlz/OWNERS (right): https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS#newcode3 rlz/OWNERS:3: gwilson@chromium.org Don't you also want to add yourself ...
8 years, 6 months ago (2012-06-22 20:08:16 UTC) #2
Roger Tawa OOO till Jul 10th
Lgtm On Jun 22, 2012 4:08 PM, <gwilson@chromium.org> wrote: > lgtm > > > https://chromiumcodereview.**appspot.com/10636014/diff/1/**rlz/OWNERS<https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS> ...
8 years, 6 months ago (2012-06-22 20:36:14 UTC) #3
Nico
Thanks! https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS File rlz/OWNERS (right): https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS#newcode3 rlz/OWNERS:3: gwilson@chromium.org On 2012/06/22 20:08:16, Glenn Wilson wrote: > ...
8 years, 6 months ago (2012-06-22 20:36:38 UTC) #4
Roger Tawa OOO till Jul 10th
8 years, 6 months ago (2012-06-22 20:36:57 UTC) #5
Lgtm
On Jun 22, 2012 4:08 PM, <gwilson@chromium.org> wrote:

> lgtm
>
>
>
https://chromiumcodereview.**appspot.com/10636014/diff/1/**rlz/OWNERS<https:/...
> File rlz/OWNERS (right):
>
> https://chromiumcodereview.**appspot.com/10636014/diff/1/**
>
rlz/OWNERS#newcode3<https://chromiumcodereview.appspot.com/10636014/diff/1/rlz/OWNERS#newcode3>
> rlz/OWNERS:3: gwilson@chromium.org
> Don't you also want to add yourself here?  Seems reasonable to me.
>
>
https://chromiumcodereview.**appspot.com/10636014/<https://chromiumcodereview...
>

Powered by Google App Engine
This is Rietveld 408576698