Show hover cards when issue numbers are followed by `)`

[ScottTodd]

We'd like to have code comments like // TODO(#123456), but a trailing space (or other whitelisted character) is needed for the hover cards to appear, like // TODO(#123456 ).

Looks like \) could be added to the final capture group here:

vscode-pull-request-github/src/issues/util.ts

Line 16 in 74e02d2

export const ISSUE_EXPRESSION = /(([^\s]+)\/([^\s]+))?#([1-9][0-9]*)($|[\s\:\;\-\(\=])/;

Not sure if there are other usage patterns that would be false positives with that character allowed.

[GMNGeoffrey]

Perhaps the final capture group be broadened to just be a word boundary (\b)?

[GMNGeoffrey]

@alexr00 can you comment on whether this would make sense (either adding \) or replacing the whole thing with \b)?

[alexr00]

I added ) to the expected characters in the regular expression.

[GMNGeoffrey]

Thanks!