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

Side by Side Diff: milo/appengine/common/model/summary.go

Issue 2944633003: [milo] Add BuildSummary and common models. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file.
4
5 package model
6
7 import "time"
8
9 // Summary summarizes a thing (step, build, group of builds, whatever).
10 type Summary struct {
11 // Status indicates the 'goodness' of the thing. This usually translates
Ryan Tseng 2017/06/17 00:44:28 On top of "goodness" It also describes where in th
iannucci 2017/06/17 01:30:04 Done.
12 // directly to a status color.
13 Status Status
14
15 // Start indicates when this thing started doing its action.
16 Start time.Time
Ryan Tseng 2017/06/17 00:44:28 Make this a pointer, nil to indicate still pending
iannucci 2017/06/17 01:30:04 Won't work in datastore. 0 value of time.Time is p
17
18 // End indicates when this thing completed doing its action.
19 End time.Time
Ryan Tseng 2017/06/17 00:44:28 Make this a pointer, to indicate no end time / sti
20
21 // Text is a possibly-multi-line summary of what happened.
22 Text []string
23 }
OLDNEW
« milo/appengine/common/model/build_summary.go ('K') | « milo/appengine/common/model/status_string.go ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698