Closed
Bug 836980
Opened 12 years ago
Closed 12 years ago
Indicate when no jobs are chosen
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sfink, Assigned: sfink)
References
Details
(Keywords: trychooser)
Attachments
(1 file, 1 obsolete file)
3.04 KB,
patch
|
emorley
:
review+
sfink
:
checked-in+
|
Details | Diff | Splinter Review |
Related to bug 836962, now that trychooser defaults to -p none, it would be good to clearly indicate when the syntax chosen will produce no jobs.
Assignee | ||
Comment 1•12 years ago
|
||
Perhaps not the most graceful implementation.
Attachment #708853 -
Flags: review?(emorley)
Comment 2•12 years ago
|
||
Comment on attachment 708853 [details] [diff] [review]
Indicate when no jobs are chosen
Thank you for doing this :-)
Since platform is our only required field, do you think we should perhaps make this specific to the platform field for now, in order that we can make the warning message clearer?
Either way, I think we should not prefix the warning message with the trychooser syntax, and display just the warning - to avoid people missing it.
Updated•12 years ago
|
Attachment #708853 -
Flags: review?(emorley)
Updated•12 years ago
|
Keywords: trychooser
Assignee | ||
Comment 3•12 years ago
|
||
This version doesn't allow you to ignore it.
Attachment #709119 -
Flags: review?(emorley)
Assignee | ||
Updated•12 years ago
|
Attachment #708853 -
Attachment is obsolete: true
Updated•12 years ago
|
Attachment #709119 -
Flags: review?(emorley) → review+
Assignee | ||
Updated•12 years ago
|
Attachment #709119 -
Flags: checked-in+
Assignee | ||
Comment 4•12 years ago
|
||
Comment 5•12 years ago
|
||
In production :-)
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
Updated•8 years ago
|
Component: Tools → General
You need to log in
before you can comment on or make changes to this bug.
Description
•