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

Issue 9677018: Add some suckers^H^H^H^H^H^H^H^Hfine upstanding project members to be OWNERS of the top-level chrom… (Closed)

Created:
8 years, 9 months ago by Dirk Pranke
Modified:
8 years, 9 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add some suckers^H^H^H^H^H^H^H^Hfine upstanding project members to be OWNERS of the top-level chrome dir. R=ben@chromium.org, jhawkins@chromium.org, jam@chromium.org, sky@chromium.org, tfarina@chromium.org, thakis@chromium.org, thestig@chromium.org BUG=117608 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126281

Patch Set 1 #

Patch Set 2 : tfarina's off the hook #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
A chrome/OWNERS View 1 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Dirk Pranke
Hi all, I am attempting to make sure all of the top-level dirs under src ...
8 years, 9 months ago (2012-03-12 22:32:09 UTC) #1
James Hawkins
LGTM
8 years, 9 months ago (2012-03-12 22:37:15 UTC) #2
Lei Zhang
On 2012/03/12 22:37:15, James Hawkins wrote: > LGTM LGTM++
8 years, 9 months ago (2012-03-12 22:58:34 UTC) #3
sky
LGTM (cringing)
8 years, 9 months ago (2012-03-12 23:36:59 UTC) #4
Ben Goodger (Google)
lgtm
8 years, 9 months ago (2012-03-12 23:38:23 UTC) #5
Nico
lgtm
8 years, 9 months ago (2012-03-12 23:41:57 UTC) #6
jam
lgtm
8 years, 9 months ago (2012-03-12 23:46:51 UTC) #7
tfarina
8 years, 9 months ago (2012-03-12 23:52:08 UTC) #8
On 2012/03/12 23:46:51, John Abd-El-Malek wrote:
> lgtm

Why I was removed? I'm ok with being out though.

Powered by Google App Engine
This is Rietveld 408576698