Change regex to accept both issue formats

[alt-13]

Closes #514 ​

Maybe not the most elegant way but my regex knowledge is limited.

Change regex to accept both issue formats:
closes https://github.com/org-name/repo/issues/123
closes org-name/repo#123

[simoneb]

Can you please describe what format this supports? Also, please include unit tests

[alt-13]

PR body is:

How else do you want to test this?
Unit test is already in place for getLinkedIssues

[alt-13]

Sry for the prettier changes. Will revert

[simoneb]

check out the unit tests in the codebase, they can serve as a blueprint about what we are testing and want to test

[simoneb]

@alt-13 just a reminder to update/add tests please

[alt-13]

@alt-13 just a reminder to update/add tests, please

I am sorry. I was unavailable last week. Please clarify if the output of the action should be seen as an external issue (owner/repo#123) as this format also can link to an external issue.
Therefore the tests were modified/created to accept issue links in the format "owner/repo#123" as external issues.

[simoneb]

Thanks for the follow up. Based on the documentation, the current output seems to be in the form "some/repo#123". In hindsight maybe that should have been a url, but since this format supports both internal and external issues, I'd say we can easily keep this format to link any issue, whether internal or extenal. What do you think?

[alt-13]

Yes, I think so too. As the documentation also states two different owner/repo combinations it is clear that it not only is linking internal but also external issues.

So for me, it is ok as it is. I hope there is no codestyle issue anywhere.

[simoneb]

please fix the linting error. you can run linting locally before pushing. can you also please undo the formatting changes to the documentation?

[alt-13]

All requested changes applied. Is there anything else to fix?