Merge lp:~brian-murray/ubuntu-archive-tools/comments-after-nag into lp:ubuntu-archive-tools

Proposed by Brian Murray on 2012-11-16
Status: Merged
Merge reported by: Scott Kitterman
Merged at revision: not available
Proposed branch: lp:~brian-murray/ubuntu-archive-tools/comments-after-nag
Merge into: lp:ubuntu-archive-tools
Diff against target: 16 lines (+6/-0)
1 file modified
sru-report (+6/-0)
To merge this branch: bzr merge lp:~brian-murray/ubuntu-archive-tools/comments-after-nag
Reviewer Review Type Date Requested Status
Scott Kitterman (community) 2012-11-16 Approve on 2012-11-16
Review via email: mp+134759@code.launchpad.net

Description of the Change

Don't turn the bug yellow / golden if there have not been comments after the verification nag, but do turn it yellow / golden if there have.

To post a comment you must log in.
Scott Kitterman (kitterman) wrote :

Merged.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'sru-report'
2--- sru-report 2012-11-16 11:51:04 +0000
3+++ sru-report 2012-11-16 23:46:21 +0000
4@@ -261,6 +261,12 @@
5 if message.date_created <= rpkg['published']:
6 continue
7 m_owner = message.owner
8+ if message.subject.lower() == \
9+ 'verification still needed':
10+ hover_text = ''
11+ if 'messages' in cls:
12+ cls = cls.replace('messages', '')
13+ continue
14 try:
15 if (m_owner not in ignored_commenters and
16 'messages' not in cls):

Subscribers

People subscribed via source and target branches