diff options
| author | Dan McGee <dan@archlinux.org> | 2011-11-03 20:39:59 -0500 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-11-03 20:39:59 -0500 |
| commit | 5f2c3bf98baabf919681525e600639643aa2c119 (patch) | |
| tree | 8738ae151d89319c2270a34ffefcfedc758bbd6b /packages/management | |
| parent | 8187b87143081a2be75032db91287f9deb9d1f89 (diff) | |
Signoffs changes and improvements
* Better signoff report with more detail
* Show signoff specification in signoffs view
* Honor disabled/bad flags and display in approval column
* Various other small bugfixes and tweaks
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'packages/management')
| -rw-r--r-- | packages/management/commands/signoff_report.py | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/packages/management/commands/signoff_report.py b/packages/management/commands/signoff_report.py index 02f3d985..3431dada 100644 --- a/packages/management/commands/signoff_report.py +++ b/packages/management/commands/signoff_report.py @@ -56,6 +56,8 @@ def generate_report(email, repo_name): # Collect all existing signoffs for these packages signoff_groups = sorted(get_signoff_groups([repo]), key=attrgetter('target_repo', 'arch', 'pkgbase')) + disabled = [] + bad = [] complete = [] incomplete = [] new = [] @@ -68,10 +70,16 @@ def generate_report(email, repo_name): old_cutoff = now - timedelta(days=old_days) for group in signoff_groups: - if group.approved(): + spec = group.specification + if spec.known_bad: + bad.append(group) + elif not spec.enabled: + disabled.append(group) + elif group.approved(): complete.append(group) else: incomplete.append(group) + if group.package.last_update > new_cutoff: new.append(group) if group.package.last_update < old_cutoff: @@ -96,6 +104,9 @@ def generate_report(email, repo_name): c = Context({ 'repo': repo, 'signoffs_url': signoffs_url, + 'disabled': disabled, + 'bad': bad, + 'all': signoff_groups, 'incomplete': incomplete, 'complete': complete, 'new': new, |