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

Issue 773813002: Remove pastarmovj from OWNERS files. (Closed)

Created:
6 years ago by pastarmovj
Modified:
6 years ago
CC:
chromium-apps-reviews_chromium.org, chromium-reviews, davemoore+watch_chromium.org, dkrahn+watch_chromium.org, extensions-reviews_chromium.org, nkostylev+watch_chromium.org, oshima+watch_chromium.org, stevenjb+watch_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Remove pastarmovj from OWNERS files. pastarmovj is leaving the Chromium project and will only preserve his contributor status but might not be able to provide timely reviews for the code he currently owns. BUG=none TEST=none Committed: https://crrev.com/f7f63a28d7d6d6759045346f03975d999e442d60 Cr-Commit-Position: refs/heads/master@{#306393}

Patch Set 1 #

Total comments: 3

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -8 lines) Patch
M chrome/browser/chromeos/attestation/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chromeos/policy/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chromeos/settings/OWNERS View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/api/enterprise_platform_keys_private/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/policy/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M chromeos/attestation/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M components/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
M components/policy/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 14 (4 generated)
pastarmovj
A sad CL =-(
6 years ago (2014-12-02 14:33:24 UTC) #2
pastarmovj
Adding jochen for components/OWNERS.
6 years ago (2014-12-02 14:34:01 UTC) #4
jochen (gone - plz use gerrit)
lgtm https://codereview.chromium.org/773813002/diff/1/chrome/browser/chromeos/settings/OWNERS File chrome/browser/chromeos/settings/OWNERS (right): https://codereview.chromium.org/773813002/diff/1/chrome/browser/chromeos/settings/OWNERS#newcode1 chrome/browser/chromeos/settings/OWNERS:1: mnissler@chromium.org one person is not enough for an ...
6 years ago (2014-12-02 14:35:16 UTC) #5
Mattias Nissler (ping if slow)
Let's wait for Yuri's answer so we can update chrome/browser/chromeos/settigns/OWNERS in one go. https://codereview.chromium.org/773813002/diff/1/chrome/browser/chromeos/settings/OWNERS File ...
6 years ago (2014-12-02 14:56:50 UTC) #7
ygorshenin1
https://codereview.chromium.org/773813002/diff/1/chrome/browser/chromeos/settings/OWNERS File chrome/browser/chromeos/settings/OWNERS (right): https://codereview.chromium.org/773813002/diff/1/chrome/browser/chromeos/settings/OWNERS#newcode1 chrome/browser/chromeos/settings/OWNERS:1: mnissler@chromium.org On 2014/12/02 14:56:50, Mattias Nissler wrote: > On ...
6 years ago (2014-12-02 15:09:11 UTC) #8
pastarmovj
Thanks Yuri, I added you to the OWNERS file.
6 years ago (2014-12-02 15:14:17 UTC) #9
Mattias Nissler (ping if slow)
LGTM 2
6 years ago (2014-12-02 15:19:51 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/773813002/20001
6 years ago (2014-12-02 15:30:22 UTC) #12
commit-bot: I haz the power
Committed patchset #2 (id:20001)
6 years ago (2014-12-02 16:10:39 UTC) #13
commit-bot: I haz the power
6 years ago (2014-12-02 16:11:30 UTC) #14
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/f7f63a28d7d6d6759045346f03975d999e442d60
Cr-Commit-Position: refs/heads/master@{#306393}

Powered by Google App Engine
This is Rietveld 408576698