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

Issue 521583008: Fix transient error regexes. (Closed)

Created:
6 years, 3 months ago by iannucci
Modified:
6 years, 3 months ago
Reviewers:
dnj, Vadim Sh., agable
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Project:
tools
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M git_common.py View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 13 (3 generated)
iannucci
6 years, 3 months ago (2014-08-29 21:45:20 UTC) #1
iannucci
On 2014/08/29 21:45:20, iannucci wrote: The error lines are actually more like "! HEAD:refs/pending/heads/master [remote ...
6 years, 3 months ago (2014-08-29 21:46:18 UTC) #2
dnj
lgtm
6 years, 3 months ago (2014-08-29 21:48:13 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/iannucci@chromium.org/521583008/1
6 years, 3 months ago (2014-08-29 22:02:37 UTC) #5
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
6 years, 3 months ago (2014-08-29 22:02:38 UTC) #7
agable
lgt
6 years, 3 months ago (2014-08-29 22:06:01 UTC) #8
agable
On 2014/08/29 22:06:01, agable wrote: > lgt m lgtm. that's what I meant.
6 years, 3 months ago (2014-08-29 22:06:12 UTC) #9
iannucci
On 2014/08/29 22:06:12, agable wrote: > On 2014/08/29 22:06:01, agable wrote: > > lgt > ...
6 years, 3 months ago (2014-08-29 22:06:21 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/iannucci@chromium.org/521583008/1
6 years, 3 months ago (2014-08-29 22:08:03 UTC) #12
commit-bot: I haz the power
6 years, 3 months ago (2014-08-29 22:10:57 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 291744

Powered by Google App Engine
This is Rietveld 408576698