Code review comment for lp:~seb128/ubuntu-archive-scripts/verdict-not-package

Revision history for this message
Iain Lane (laney) wrote :

Ah yeah, this works I guess but you could also delete those entries before handing the dictionary to the template, that might be cleaner? up to you, something like this (not tested!):

=== modified file 'generate-team-p-m'
--- old/generate-team-p-m 2020-04-07 10:02:51 +0000
+++ new/generate-team-p-m 2020-06-30 15:06:56 +0000
@@ -256,6 +256,11 @@
         in_proposed_packages[source_package_name] = prob
         prob.regressions = []
         prob.waiting = []
+ for policy in ['autopkgtest', 'update-excuse', 'block-bugs']:
+ try:
+ del item['policy_info'][policy]['verdict']
+ except KeyError:
+ pass
         if 'autopkgtest' in item['reason']:
             for package, results in sorted(item['policy_info']['autopkgtest'].items()):
                 regr_arches = []

« Back to merge proposal