New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 665580 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Migrate Skia to PolyGerrit

Project Member Reported by aga...@chromium.org, Nov 15 2016

Issue description

Comment 1 by aga...@chromium.org, Nov 15 2016

Status: Started (was: Assigned)
This has already gone most of the way. All repos have Gerrit enabled behind a flag, and both skia and buildbot have been switched to use PG by default.

Next steps:
* Switch remaining three repos to use PG by default

Comment 2 by aga...@chromium.org, Nov 15 2016

Cc: borenet@chromium.org djsollen@chromium.org rmis...@chromium.org
Here are the three CLs:

https://skia-review.googlesource.com/c/4876/
https://skia-review.googlesource.com/c/4877/
https://skia-review.googlesource.com/c/4878/

Note that you can't see the last one in an incognito window, as desired.
LGTM!

Comment 4 by rmis...@google.com, Nov 16 2016

LGTM as well.
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 17 2016

The following revision refers to this bug:
  https://skia.googlesource.com/common.git/+/7c44d925ea329b093cbd259113308bcba8b7804b

commit 7c44d925ea329b093cbd259113308bcba8b7804b
Author: Aaron Gable <agable@chromium.org>
Date: Tue Nov 15 23:22:29 2016

Switch skia common to use PolyGerrit by default

BUG= chromium:665580 

Change-Id: I74b4c2be61a383cf16efaa496a757afed6178687
Reviewed-on: https://skia-review.googlesource.com/4876
Reviewed-by: Eric Boren <borenet@google.com>

[modify] https://crrev.com/7c44d925ea329b093cbd259113308bcba8b7804b/codereview.settings

Comment 6 by aga...@chromium.org, Nov 22 2016

Status: Fixed (was: Started)

Sign in to add a comment