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

Issue 10392021: Remove evan[m]?@chromium.org from OWNERS files. (Closed)

Created:
8 years, 7 months ago by tony
Modified:
8 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, pam+watch_chromium.org
Visibility:
Public.

Description

Remove evan[m]?@chromium.org from OWNERS files. He no longer works on this project. BUG=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=136374

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -3 lines) Patch
M content/app/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M tools/checklicenses/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M ui/base/gtk/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
tony
8 years, 7 months ago (2012-05-09 23:55:06 UTC) #1
Evan Martin
LGTM On Wed, May 9, 2012 at 4:55 PM, <tony@chromium.org> wrote: > Reviewers: Evan Martin, ...
8 years, 7 months ago (2012-05-10 16:50:00 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tony@chromium.org/10392021/1
8 years, 7 months ago (2012-05-10 18:21:46 UTC) #3
commit-bot: I haz the power
Presubmit check for 10392021-1 failed and returned exit status 1. Running presubmit commit checks ...
8 years, 7 months ago (2012-05-10 18:21:53 UTC) #4
Evan Martin
HAHAHAHA On Thu, May 10, 2012 at 11:21 AM, <commit-bot@chromium.org> wrote: > Presubmit check for ...
8 years, 7 months ago (2012-05-10 18:25:20 UTC) #5
tony
8 years, 7 months ago (2012-05-10 20:22:14 UTC) #6
Meh, I just forced the commit.

On 2012/05/10 18:25:20, Evan Martin wrote:
> HAHAHAHA
> 
> On Thu, May 10, 2012 at 11:21 AM,  <mailto:commit-bot@chromium.org> wrote:
> > Presubmit check for 10392021-1 failed and returned exit status 1.
> >
> > Running presubmit commit checks ...
> >
> > ** Presubmit Messages **
> > If this change requires manual test instructions to QA team, add
> > TEST=[instructions].
> >
> > ** Presubmit ERRORS **
> > Missing LGTM from an OWNER for files in these directories:
> > &nbsp; &nbsp;content/app
> > &nbsp; &nbsp;ui/base/gtk
> > &nbsp; &nbsp;tools/checklicenses
> >
> > Presubmit checks took 1.1s to calculate.
> >
> > Was the presubmit check useful? Please send feedback & hate mail to
> > maruel@chromium.org!
> >
> >
> > https://chromiumcodereview.appspot.com/10392021/

Powered by Google App Engine
This is Rietveld 408576698