Merge lp:~vorlon/ubuntu-archive-scripts/better-team-report into lp:ubuntu-archive-scripts

Proposed by Steve Langasek
Status: Merged
Merged at revision: 237
Proposed branch: lp:~vorlon/ubuntu-archive-scripts/better-team-report
Merge into: lp:ubuntu-archive-scripts
Diff against target: 37 lines (+10/-10)
1 file modified
templates/team-report.html (+10/-10)
To merge this branch: bzr merge lp:~vorlon/ubuntu-archive-scripts/better-team-report
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle (community) Approve
Review via email: mp+365705@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Makes sense. Can't actually mark this approved it seems.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'templates/team-report.html'
2--- templates/team-report.html 2019-02-04 21:38:10 +0000
3+++ templates/team-report.html 2019-04-08 23:03:40 +0000
4@@ -45,16 +45,6 @@
5 {% endif %}
6 {{ ngettext("%(num)d day", "%(num)d days", prob.age|int) }}
7 <ul>
8- {% if prob.waiting %}
9- <li>
10- Waiting
11- <ul>
12- {% for p in prob.waiting %}
13- <li>{{ p }}</li>
14- {% endfor %}
15- </ul>
16- </li>
17- {% endif %}
18 {% if d.get("missing-builds") %}
19 <li>Missing builds, see excuses</li>
20 {% endif %}
21@@ -77,6 +67,16 @@
22 </ul>
23 </li>
24 {% endif %}
25+ {% if prob.waiting %}
26+ <li>
27+ Waiting
28+ <ul>
29+ {% for p in prob.waiting %}
30+ <li>{{ p }}</li>
31+ {% endfor %}
32+ </ul>
33+ </li>
34+ {% endif %}
35 {% if prob.kind != "regressing-other" %}
36 {% if d.get('dependencies', {}).get('blocked-by', []) %}
37 <li>Depends on {{ d['dependencies']['blocked-by']|join(", ") }}</li>

Subscribers

People subscribed via source and target branches