Code review comment for lp:~gmb/maas/fix-bug-1402237-1.7

Revision history for this message
Raphaël Badin (rvb) wrote :

Looks good. Couple of remarks below but nothing major.

[0]

Typo in the commit message: fecthing

[1]

Could you say in the commit message why we're using this trick instead of adding the index… and that the index is being added to trunk…?

review: Approve

« Back to merge proposal