-
Notifications
You must be signed in to change notification settings - Fork 210
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
Update reference links #67
Conversation
Done by a "replace all" - to be reviewed!
Some links had not been captured by the first replace-all pass. Some anchors used a different pattern on GitHub than they use on the Khronos registry page.
@emackey This is the third of three pull requests They build on top of each other: The first ones are rather simple and small, and this one is rather technical (in that it mainly consisted of a "replace all"), so I thought that it wouldn't make much sense to have them as separate PRs, always starting from Depending on the preferred process, it could make sense to review and merge this one, implicitly dropping the two other ones. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few URL tweaks, but this is great @javagl. Thanks!
As suggested in the PR review Co-authored-by: Ed Mackey <[email protected]>
Thanks @emackey ! I had done a "full text search", and a largely automated replace, with manual tweaks for some critical ones (e.g. some of these |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a big improvement. I'm going to merge it now and then do another pass looking for any stragglers.
To be merged after #66
Fixes #59