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

Issue 2338263003: Remove steel@ from audio OWNERS. (Closed)

Created:
4 years, 3 months ago by Rahul Chaturvedi
Modified:
4 years, 3 months ago
Reviewers:
stevenjb, jennyz
CC:
chromium-reviews, hashimoto+watch_chromium.org, oshima+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove steel@ from audio OWNERS. I haven't worked with audio in a while and am now missing enough context that I shouldn't be on the owners files. Removing myself. Need stevenjb's lgtm for removing me from //chromeos/dbus/OWNERS. R=jennyz@chromium.org, stevenjb@chromium.org BUG=None. Committed: https://crrev.com/548f334b7adaa33c6a3c2fa8af8eee629d88d0b0 Cr-Commit-Position: refs/heads/master@{#418724}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M chromeos/audio/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chromeos/dbus/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
Rahul Chaturvedi
4 years, 3 months ago (2016-09-14 23:16:01 UTC) #1
stevenjb
lgtm
4 years, 3 months ago (2016-09-14 23:21:47 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2338263003/1
4 years, 3 months ago (2016-09-14 23:24:59 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-15 00:31:37 UTC) #5
commit-bot: I haz the power
4 years, 3 months ago (2016-09-15 00:34:10 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/548f334b7adaa33c6a3c2fa8af8eee629d88d0b0
Cr-Commit-Position: refs/heads/master@{#418724}

Powered by Google App Engine
This is Rietveld 408576698