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

Issue 1455883005: Remove WebKit's validate-committer-lists script. (Closed)

Created:
5 years, 1 month ago by Nico
Modified:
5 years, 1 month ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, blink-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove WebKit's validate-committer-lists script. We haven't used WebKit's committer list for anything in a long time. BUG=none Committed: https://crrev.com/cd0ab74ff27c0449592fb99cbeb987784010f194 Cr-Commit-Position: refs/heads/master@{#360588}

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -287 lines) Patch
D third_party/WebKit/Tools/Scripts/validate-committer-lists View 1 chunk +0 lines, -287 lines 1 comment Download

Messages

Total messages: 8 (2 generated)
Nico
last one
5 years, 1 month ago (2015-11-19 05:45:24 UTC) #2
Nico
https://codereview.chromium.org/1455883005/diff/1/third_party/WebKit/Tools/Scripts/validate-committer-lists File third_party/WebKit/Tools/Scripts/validate-committer-lists (left): https://codereview.chromium.org/1455883005/diff/1/third_party/WebKit/Tools/Scripts/validate-committer-lists#oldcode42 third_party/WebKit/Tools/Scripts/validate-committer-lists:42: from webkitpy.common.system.deprecated_logging import log, error (none of these modules ...
5 years, 1 month ago (2015-11-19 05:47:31 UTC) #3
Dirk Pranke
lgtm
5 years, 1 month ago (2015-11-19 08:49:36 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1455883005/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1455883005/1
5 years, 1 month ago (2015-11-19 15:33:21 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-11-19 15:45:58 UTC) #7
commit-bot: I haz the power
5 years, 1 month ago (2015-11-19 15:48:21 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/cd0ab74ff27c0449592fb99cbeb987784010f194
Cr-Commit-Position: refs/heads/master@{#360588}

Powered by Google App Engine
This is Rietveld 408576698