Code review comment for lp:~leonardr/launchpad/use-web-link

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

I can't say I like the "&&" at the beginning of a line, but I won't force my tastes on you. Otherwise, looks fine.

As per IRC, we're not foreseeing any legitimate scenario under which web_link might not be present, so the one obvious path of breakage is not to be expected.

Jeroen

review: Approve (code)

« Back to merge proposal