I still need to review the tests, but I have the following comments (as discussed in our 1 on 1):
In trigger_ci_build() and trigger_al_build(), the if launchpadutils.testing_in_progress(mp, jenkins_job): continue
check should be the last check before proceeding to the trigger as it is likely the most time and resource intensive check.
« Back to merge proposal
I still need to review the tests, but I have the following comments (as discussed in our 1 on 1):
In trigger_ci_build() and trigger_al_build(), the testing_ in_progress( mp, jenkins_job):
if launchpadutils.
continue
check should be the last check before proceeding to the trigger as it is likely the most time and resource intensive check.