comprehensive-rust icon indicating copy to clipboard operation
comprehensive-rust copied to clipboard

Check links are correct and fix those that aren't

Open adetaylor opened this issue 2 years ago • 3 comments

Fixes #957

adetaylor avatar Nov 28 '23 20:11 adetaylor

Thanks @adetaylor! I rebased this on top of main and there are a few new errors — in addition to the ones in the translations.

You could update the translations yourself, we could disable checking links there, or perhaps some other approach?

Testing this out it's nice: it caches the results and won't slow down the build (or mdbook serve).

mgeisler avatar Nov 30 '23 14:11 mgeisler

You could update the translations yourself, we could disable checking links there, or perhaps some other approach?

It actually looked to be a little fiddly to update the translations, since it appears that they do link to pages which may (or may not) actually exist when your browser is set to a different language. I think it possibly does need the actual translators to go through the mistakes.

I'm not quite sure what to do here. Disabling the link checking for translations seems bad, but perhaps as an interim step we should do that?

adetaylor avatar Nov 30 '23 16:11 adetaylor

@adetaylor I'm in favor of disabling broken link checking for translations 👍 At least ensuring that the link is correct in the original English is an improvement of the current situation, and we can continue to fix broken links in the translations manually until we can figure out how to automatically valid links in translations.

Can you rebase this onto main and update it to only check the English slides?

randomPoison avatar Jan 22 '24 18:01 randomPoison

Hi, please check out #2195 which does the same and more on top of the current main.

jond01 avatar Jul 17 '24 17:07 jond01

@mgeisler I think this can be closed now as #2195 has been merged.

jond01 avatar Jul 23 '24 20:07 jond01

Yes, let us close this. Thanks @adetaylor for starting this and thanks for pushing this over the finishing line, @jond01!

mgeisler avatar Aug 08 '24 19:08 mgeisler