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

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

Brendan, I need the global just to fix the setting of _JENKINS_API, i.e. your code appears to require the global declaration also. We would add the leading underscore if the use of these global constants (in settings files) was not already rampant in Django :) . (Offering this paste as demonstration: http://pastebin.ubuntu.com/7496861/ .)

« Back to merge proposal