Code review comment for ~sespiros/ubuntu-cve-tracker/+git/ubuntu-cve-tracker:source-map-fix

Revision history for this message
Alex Murray (alexmurray) wrote (last edit ):

Thanks - but now I hate the fact this function is named `release_has_parter()` but takes an argument called `series` - and then always adds a `ubuntu/` prefix which seems too presumptuous - sorry for the run-around - could we instead do something like:

def release_has_partner(rel):
    try:
      _, _, _, details = get_subproject_details(rel)
      return "partner" in details["components"]
    except (ValueError, KeyError):
      return False

« Back to merge proposal