In this case I probably could have said:

```
return issue.pull_request && issue.pull_request.html_url;
```
I just didn't like the way it would allow things through that may not be actual URLs.  (ex: numbers)  That said, the value should either be a URL or undefined/null.  So it *should* work.