Code review comment for lp:~oif-team/grail/cooperate-with-xi2.1

Revision history for this message
Chase Douglas (chasedouglas) wrote :

I liked the stylistic simplicity of having the consts within the function they are used. Keeping things in scope. But I also agree that it's nice having constants defined at the top. I'll move them up there when I merge it tomorrow.


« Back to merge proposal