Code review comment for lp:~allanlesage/qa-coverage-dashboard/jenkinsapi-singleton

Revision history for this message
Allan LeSage (allanlesage) wrote :

Discussed with thomi and ChrisGagnon; thomi demonstrated the use of functools.partial but I don't want to use that here because it requires instantiating the jenkinsapi.Jenkins at a level above, and I don't want the top-level code to have to know about Jenkins. Instead I'm hiding the instantiated singleton in a function attribute as thomi also demonstrated.

« Back to merge proposal