Merge lp:~brian-murray/ubuntu-archive-tools/gray-removals into lp:ubuntu-archive-tools

Proposed by Brian Murray
Status: Merged
Merged at revision: 694
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/gray-removals
Merge into: lp:ubuntu-archive-tools
Diff against target: 40 lines (+8/-2)
1 file modified
sru-report (+8/-2)
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/gray-removals
Reviewer Review Type Date Requested Status
Colin Watson Approve
Review via email: mp+158502@code.launchpad.net

Description of the change

If a bug is tagged removal-candidate (not been verified for more than 90 days) make it gray colored so that we know which packages may be removed.

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sru-report'
--- sru-report 2013-02-14 10:51:38 +0000
+++ sru-report 2013-04-11 22:17:20 +0000
@@ -161,6 +161,7 @@
161 a.kerneltracking { font-style: italic; }161 a.kerneltracking { font-style: italic; }
162 a.testing { color: blue; }162 a.testing { color: blue; }
163 a.broken { text-decoration: line-through; color: black; }163 a.broken { text-decoration: line-through; color: black; }
164 a.removal { color: gray; font-weight: bold }
164 </style>165 </style>
165</head>166</head>
166<body>167<body>
@@ -189,6 +190,8 @@
189but need verification for another task,190but need verification for another task,
190bugs in <span style="color: #999900;">golden</span> have received a comment191bugs in <span style="color: #999900;">golden</span> have received a comment
191since the package was accepted in -proposed,192since the package was accepted in -proposed,
193bugs in <span style="color: gray;">gray</span> are candidates for removal
194due to a lack of verification,
192bugs in <span style="font-style: italic">italic</span> are kernel tracking195bugs in <span style="font-style: italic">italic</span> are kernel tracking
193bugs and bugs that are196bugs and bugs that are
194<span style="text-decoration: line-through;">struck through</span> weren't197<span style="text-decoration: line-through;">struck through</span> weren't
@@ -279,14 +282,17 @@
279 minute=0, second=0, microsecond=0)282 minute=0, second=0, microsecond=0)
280 published_date = rpkg['published'].replace(283 published_date = rpkg['published'].replace(
281 minute=0, second=0, microsecond=0)284 minute=0, second=0, microsecond=0)
285 today = datetime.datetime.utcnow()
282 if last_message_date > published_date:286 if last_message_date > published_date:
283 for message in bug.messages:287 for message in bug.messages:
284 m_date = message.date_created288 m_date = message.date_created
285 if m_date <= rpkg['published']:289 if m_date <= rpkg['published']:
286 continue290 continue
287 m_owner = message.owner291 m_owner = message.owner
288 if message.subject.lower() == \292 if 'verification still needed' in message.subject.lower():
289 'verification still needed':293 if m_date.replace(tzinfo=None) < today - datetime.timedelta(16):
294 cls += ' removal'
295 continue
290 hover_text = ''296 hover_text = ''
291 if 'messages' in cls:297 if 'messages' in cls:
292 cls = cls.replace('messages', '')298 cls = cls.replace('messages', '')

Subscribers

People subscribed via source and target branches