Code review comment for ~pfsmorigo/review-tools:issues_email

Revision history for this message
Alex Murray (alexmurray) wrote :

LGTM but I wonder if using linkskey in ["xxx", "yyy"] might be easier to read than linkskey == "xxx" or linkskey = "yyy" etc (plus it is easier to just add another case in the future by adding it to the list as well)

review: Approve

« Back to merge proposal