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

Issue 9665022: Add an OWNERS file for sdch (Closed)

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

Description

Add an OWNERS file for sdch. As part of the changes to make sure all directories have proper lists of owners, I'm adding one here ... R=jar@chromium.org BUG=88315 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126283

Patch Set 1 #

Total comments: 1

Patch Set 2 : update w/ review feedback #

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

Messages

Total messages: 8 (0 generated)
Dirk Pranke
I'm kinda guessing ... let me know if there are better owners for this.
8 years, 9 months ago (2012-03-09 23:07:35 UTC) #1
willchan no longer on Chromium
No, I shouldn't be in there, unless we need more than just jar@ (I believe ...
8 years, 9 months ago (2012-03-09 23:55:44 UTC) #2
Dirk Pranke
On 2012/03/09 23:55:44, willchan wrote: > No, I shouldn't be in there, unless we need ...
8 years, 9 months ago (2012-03-12 20:51:47 UTC) #3
wtc
dpranke: we pull most of the files in sdch directly from open-vcdiff upstream. Our own ...
8 years, 9 months ago (2012-03-12 21:57:40 UTC) #4
jar (doing other things)
lgtm
8 years, 9 months ago (2012-03-12 21:58:48 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dpranke@chromium.org/9665022/1
8 years, 9 months ago (2012-03-12 21:59:03 UTC) #6
Dirk Pranke
On 2012/03/12 21:57:40, wtc wrote: > dpranke: we pull most of the files in sdch ...
8 years, 9 months ago (2012-03-12 22:04:50 UTC) #7
wtc
8 years, 9 months ago (2012-03-12 22:07:51 UTC) #8
Patch set 2 LGTM.  Thanks!

Powered by Google App Engine
This is Rietveld 408576698