Change existing HTMLs to redirect to github pages #308
Conversation
The patch rewrite all extension to redirect to the github pages location. The README now ask to no longer publish the HTML along side the document.
|
If you want to test how this behave: https://htmlpreview.github.io/?https://github.com/Naghasan/SPIRV-Registry/blob/redirect/extensions/AMD/SPV_AMD_gcn_shader.html |
There was a problem hiding this comment.
LGTM. I didn't check every file, but I did spot check several, and everything looked in order.
I also checked the "back" button after a redirect, and at least with my browser it went back to the proper page and didn't redirect again.
A few things to confirm:
-
We merged a couple of extensions last week (add SPV_INTEL_2d_block_io #305 and add SPV_INTEL_subgroup_matrix_multiply_accumulate #306) with included HTML specifications. This PR should either update them to include the redirect too, or it should delete them. I have a slight preference to update them, just in cases any links to them have leaked out in the last week, but I'd be fine either way.
-
There is a merge conflict now due to Fix typo in QCOM capability #307 that will need to be resolved before merging.
|
I updated the branch and regenerated the redirect files, that includes the extensions added last week. |
|
Merging as discussed in the January 22nd teleconference. |
The patch rewrite all extension to redirect to the github pages location. The README now ask to no longer publish the HTML along side the document.