-
-
Notifications
You must be signed in to change notification settings - Fork 136
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 Graal url to run JS #1290
📝 Fix Graal url to run JS #1290
Conversation
Also found this other urls: no sure which one you wanted to use: |
Sigh 😞
Thanks! |
019824d
to
2c27267
Compare
2c27267
to
681f758
Compare
It's more a build issue. I know the site somehow checks links with https://github.com/asciidoctor/docs.asciidoctor.org/blob/main/checklinks.js. At least I've seen the messages when doing PRs on docs.asciidoctor.org, but we don't get any notification when there's a change in the contents repo. I wonder if there's at least a log we can check in netlify.... |
I said "sigh" because breaking documentation URL is bad! I'm a bit disappointed by GraalVM, they didn't bother to redirect https://www.graalvm.org/docs/graalvm-as-a-platform/embed/ to https://graalvm.org/reference-manual/embed-languages
I don't think we check external links? Do we?
|
Seems the url to the Graal docs has changed.
This PR just updates it to avoid a 404.