Skip to content

Conversation

@kevinoid
Copy link
Contributor

@kevinoid kevinoid commented Jan 3, 2021

Thanks for maintaining the octokit modules, they are quite impressive!

While I was looking around, I noticed that the link to "GitHub's OpenAPI spec" in src/generated/README.md is dead. The cause is that, without a protocol, this link is interpreted as a relative URL. If the file is viewed at https://github.com/octokit/plugin-rest-endpoint-methods.js/blob/master/src/generated/README.md, the link resolves to https://github.com/octokit/plugin-rest-endpoint-methods.js/blob/master/src/generated/github.com/github/rest-api-description which does not exist. This PR adds https:// so the link resolves to https://github.com/github/rest-api-description which I assume is the original intention.

Thanks,
Kevin

Without a protocol, this link is interpreted as a relative URL, so if
the file is viewed at
https://github.com/octokit/plugin-rest-endpoint-methods.js/blob/master/src/generated/README.md
the link resolves to
https://github.com/octokit/plugin-rest-endpoint-methods.js/blob/master/src/generated/github.com/github/rest-api-description
which does not exist.  This commit adds the https:// protocol so the
link always resolve to https://github.com/github/rest-api-description

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
@wolfy1339 wolfy1339 added Type: Documentation Improvements or additions to documentation Type: Bug Something isn't working as documented labels Jan 4, 2021
Copy link
Member

@wolfy1339 wolfy1339 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@gr2m gr2m merged commit 00c07cc into octokit:master Jan 4, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2021

🎉 This PR is included in version 4.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug Something isn't working as documented Type: Documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants