Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(640)

Issue 75630043: code review 75630043: go.tools/dashboard/app: send log hash to gobot on build... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 2 months ago by adg
Modified:
11 years, 2 months ago
Reviewers:
gobot, rsc
CC:
rsc, golang-codereviews
Visibility:
Public.

Description

go.tools/dashboard/app: send log hash to gobot on build failure Fixes issue 7322.

Patch Set 1 #

Patch Set 2 : diff -r 293a3ab6409d https://code.google.com/p/go.tools #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -1 line) Patch
M dashboard/app/build/notify.go View 1 chunk +6 lines, -1 line 0 comments Download

Messages

Total messages: 8
adg
Hello rsc (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go.tools
11 years, 2 months ago (2014-03-13 21:37:21 UTC) #1
rsc
LGTM
11 years, 2 months ago (2014-03-13 22:49:23 UTC) #2
adg
*** Submitted as https://code.google.com/p/go/source/detail?r=57bd0f2372e7&repo=tools *** go.tools/dashboard/app: send log hash to gobot on build failure Fixes ...
11 years, 2 months ago (2014-03-13 22:51:20 UTC) #3
adg
This is deployed. The ball's in your gobot court now, Russ.
11 years, 2 months ago (2014-03-13 22:52:33 UTC) #4
gobot
This CL appears to have broken the testing builder.
11 years, 2 months ago (2014-03-13 23:01:12 UTC) #5
gobot
This CL appears to have broken the testing builder.
11 years, 2 months ago (2014-03-13 23:01:50 UTC) #6
gobot
This CL appears to have broken the testing builder.
11 years, 2 months ago (2014-03-13 23:02:38 UTC) #7
gobot
11 years, 2 months ago (2014-03-13 23:14:43 UTC) #8
Message was sent while issue was closed.
This CL appears to have broken the testing builder.
See http://build.golang.org/log/quux
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b