Merge lp:~brian-murray/ubuntu-archive-tools/change-purple into lp:ubuntu-archive-tools

[Brian Murray]

07:57 < slangasek> bdmurray: should the pending-sru report maybe
                   show a bug as green instead of purple when it's
                   verified for *this* series that we're currently
                   looking at?
07:59 < bdmurray> slangasek: because now if its purple you won't
                  look at the bug, but if it's green you would?
08:01 < slangasek> bdmurray: if I have a mix of green and purple I
                   have to look at the bug to see if it's fixed for
                   this release; if it's all green I know it should
                   be ready to release and can just load up all the
                   bugs for one final check
08:03 < bdmurray> slangasek: ack, so it'd help with multi-bug SRUs.
08:04 < slangasek> bdmurray: imho yes

[Steve Langasek (vorlon)]

I'm confused by the changes to both the text and the code. See comments inline.

review: Needs Information

[Brian Murray (brian-murray)]

I've inline replied to your inline comment.

[lp:~brian-murray/ubuntu-archive-tools/change-purple]

1093. By Brian Murray on 2017-04-19

drop the verification partial logic, also the # in front of the bug number serves no purpose.

[Brian Murray (brian-murray)]

I've dropped the verification partial classification. I also removed the # from in front of the bug number because it seems silly.

[lp:~brian-murray/ubuntu-archive-tools/change-purple]

1094. By Brian Murray on 2017-04-19

finish removing the purple refernce

=== modified file 'sru-report'
--- sru-report	2017-04-13 22:17:50 +0000
+++ sru-report	2017-04-19 20:57:10 +0000
@@ -186,7 +186,6 @@
     a.messages { color: #999900; font-weight: bold; }
     a.verified { color: green; font-weight: bold; }
     a.verificationfailed { color: red; font-weight: bold; }
-    a.verificationpartial { color: #660099; font-weight: bold; }
     a.kerneltracking { font-style: italic; }
     a.testing { color: blue; }
     a.broken { text-decoration: line-through; color: black; }
@@ -218,10 +217,8 @@
 href="http://people.canonical.com/~ubuntu-archive/phased-updates.html">
 report</a>.</p>
 
-<p>Bugs in <span style="color:green;">green</span> are verified by QA,
+<p>Bugs in <span style="color:green;">green</span> are verified,
 bugs in <span style="color:red;">red</span> failed verification,
-bugs in <span style="color:#660099;">purple</span> are verified for a task
-but need verification for another task,
 bugs in <span style="color: #999900;">golden</span> have received a comment
 since the package was accepted in -proposed,
 bugs in <span style="color: gray;">gray</span> are candidates for removal
@@ -338,11 +335,6 @@
                     cls += 'kerneltracking '
                 if 'verification-failed' in t:
                     cls += ' verificationfailed'
-                elif (('verification-done' in t or
-                       'verification-done-%s' % release in t) and
-                      'verification-needed' in t):
-                    cls += ' verificationpartial'
-                    removable = False
                 elif ('verification-done' in t or
                       'verification-done-%s' % release in t):
                     cls += ' verified'
@@ -395,7 +387,7 @@
                 cls += '"'
 
                 print('<a href="%s/bugs/%d" '
-                      'title="%s" %s>#%d%s</a>' %
+                      'title="%s" %s>%d%s</a>' %
                       (lp_url, b, hover_text.replace('"', ''), cls, b,
                        '(hw)' if 'hw-specific' in t else ''))
             if antique and removable: