docs(PULL_REQUEST_TEMPLATE.md): fix broken links in PR template

[frederikprijck]

fix broken links in PR template

[frederikprijck]

I'm not sure if hardcoding blob/master is a good idea. But I prefer a working link instead of a broken. 😬

[gkalpak]

Can you change the commit message to docs(PULL_REQUEST_TEMPLATE.md): ...?

[frederikprijck]

@gkalpak Updated! :)

[Narretz]

Can you please change this and the other link below to be relative? ../blob/master/DEVELOPERS.md#commits This way, they work the same in forks.

[frederikprijck]

Mmm this doesn't seem to work in every situation, using an absolute url does.

I did the following:

• Edit the template file using github UI
• Update the URL to reflect ur changes
• Preview the file
• Click the link to test if it works and see it won't work coz it tries to navigate to https://github.com/frederikprijck/angular.js/blob/patch-2/blob/master/DEVELOPERS.md#commits

@Narretz Am I missing something? And what's the benefit of using a relative link to a hardcoded branch over an absolute one to the same branch?

Tbh I'd would love to have a solution which points to the file in the correct branch. However, I'm not sure that's worth the effort as there's other hardcoded absolute urls in the repo.

[Narretz]

Ah, okay the relative format only works well if you are in the repository view / file tree. It doesn't work as part of the PR template. Just leave it absolute.

[frederikprijck]

I think we can use /Developers.md!

Edit: Nvm, doesn't seem to behave the same in all situations.