Code review comment for lp:~exsdev/landscape-client-charm/juju-app-annotator

Revision history for this message
Edin S (exsdev) wrote :

Next reviewer: please decide if line 61 should be changed. It's potentially inefficient in that it makes two calls to regex.match(agent) for every match, but it feels cleaner than the alternatives: a ~5 line for-loop or a double-nested list-comprehension.

« Back to merge proposal