Code review comment for lp:~free.ekanayaka/charms/trusty/landscape-client/machine-id-charm-info

Revision history for this message
Alberto Donato (ack) wrote :

Looks good, +1

#1:
+ return match[(len(pattern) - 1):].replace("-", "/")

Please add a comment for this line, it took me a bit to understand the reason for the replace.

review: Approve

« Back to merge proposal