-
Notifications
You must be signed in to change notification settings - Fork 51
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
CRD publication fails due to "wrong-w3c-state-link" error #1887
Comments
@deniak this looks like a Link Checker and Specberus interop issue. Which one is right? The former passes while the latter fails. If Link Checker is correct, I think a Specberus fix might be: - const expectedLink = `https://www.w3.org/standards/types#${hash}`;
+ const expectedLink = `https://www.w3.org/standards/types/#${hash}`;
specberus/lib/rules/headers/w3c-state.js Line 58 in 1e009e0
|
I just finally debugged to figure out this is why WebGPU's publish-to-TR step is failing. IMO the version with the Would love to see this fixed! I would open a PR with Anssi's fix, however I don't know if simply updating that line will break a bunch of other projects? |
Just re-ran the publish job for https://www.w3.org/TR/webgpu/ and it worked! Thank you! |
Similarly, I can confirm https://www.w3.org/TR/webnn/ publish workflow is green again. Thanks a bunch @deniak! I'll close this issue. |
@deniak Specberus publication started failing with wrong-w3c-state-link error. It looks the trailing slash at the end of the w3c-state-link URL is the culprit. However, we haven't changed our workflow or bikeshed-boilerplate. Failure first noticed after this commit: webmachinelearning/webnn@d1d71ed
TR fails to deploy: https://www.w3.org/TR/webnn/
ED builds properly: https://webmachinelearning.github.io/webnn/
Your Specberus-fu welcome!
The text was updated successfully, but these errors were encountered: