Code review comment for ~fourdollars/plainbox-provider-checkbox:master

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Job looks good to me too and indeed it does not take too long to run so it could be a good candidate to run as part of:

https://git.launchpad.net/plainbox-provider-checkbox/tree/units/miscellanea/test-plan.pxu#n21

Moreover debsums is not listed in the LTS manifests, so please create the following packaging meta data unit to add the dependency (in a units/miscellanea/packaging.pxu):

unit: packaging meta-data
os-id: debian
Depends: debsums

review: Needs Fixing

« Back to merge proposal