Code review comment for lp://staging/~vila/ubuntu-ci-services-itself/secgroups

Revision history for this message
Andy Doan (doanac) wrote :

19 + # FIXME: uci-engine-ci is a bit dry and needs some help. Postponing
20 + # full isolated setup for now -- vila 2014-02-07

this is a bit subjective from me. but do we really need this comment in code? ie what's the long term idea you have for ensuring the fact that calls to bzr won't need BZR_EMAIL setup? I just fear comments like this, wind up in code and then live forever causing noise. maybe its just a better explanation is needed in the FIXME so i understand what you are envisioning.

« Back to merge proposal