Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix links in PULL_REQUEST_TEMPLATE #524

Merged
merged 3 commits into from Apr 7, 2017
Merged

Conversation

adiorion
Copy link
Contributor

@adiorion adiorion commented Apr 5, 2017

fix internal doc links in PRs

The links in PRs such as #517 don't go to the relevant docs, they just say "Not Found". I don't know how to link from a PR to "a given file in in the current branch" so this PR updates them to point to the master branch.

Unfortunately this means that the links are broken in the file view https://github.com/poteto/hiring-without-whiteboards/blob/master/.github/PULL_REQUEST_TEMPLATE.md but I don't think it's possible to get both given current Github limitations?

The links in PRs such as poteto#517 don't go to the relevant docs. I don't know how to link from a PR to "a given file in in the current branch" so this updates them to point to the `master` branch.
@adiorion adiorion mentioned this pull request Apr 5, 2017
4 tasks
@kethinov
Copy link
Contributor

kethinov commented Apr 5, 2017

I opened a trivial (pointless) PR to test this. Looks like it still doesn't work right. Using absolute URLs seems to work though.

@kethinov
Copy link
Contributor

kethinov commented Apr 5, 2017

Oh duh looks like this isn't merged yet. No wonder it didn't work :O

@kethinov
Copy link
Contributor

kethinov commented Apr 5, 2017

Yep, it does appear to work. Closed my test PR.

@poteto
Copy link
Owner

poteto commented Apr 5, 2017

Could you use an absolute link instead please? e.g. https://github.com/poteto/hiring-without-whiteboards/blob/master/CONTRIBUTING.md

@adiorion
Copy link
Contributor Author

adiorion commented Apr 5, 2017

@poteto good idea! lemme push that

@poteto poteto merged commit a093db8 into poteto:master Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants