Code review comment for ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/reason_in_notes

Revision history for this message
Paulo Flabiano Smorigo (pfsmorigo) wrote :

> Leaving one question just in case, otherwise LGTM!

Fixed the mistypo and added a check as you suggested. I added it anyway but, to be honest, the check is not really necessary since the element is always a tuplet with two elements:

>>> cveinfo["CVE-2019-11840"]["Priority"]
['medium', '']
>>> cveinfo["CVE-2019-11840"]["Priority_snapd"]
['negligible', '']
>>> cveinfo["CVE-2007-6752"]["Priority"]
['low', '']

« Back to merge proposal