Issues with `markdown-link-check-disable-next-line`
in index.mdx within /docs/build-on-linea/linea-version, the markdown link check escape tags are throwing errors.
I'm going to comment them out, for now. This seems to be a misalignment between tooling, ie, how does the link checker parse HTML in an MDX context.
I think it might have been me who deliberately added these. In some cases it's the only way to get the link linter to pass. The root cause is the linter, as since it throws errors for functional links, we're left with two choices:
- Ignore the failed linter, double check the links it identifies as broken do actually work, and merge anyway
- Add these ^ commands to pages that throw errors to enable the linter to pass, but break the formatting.
Yeah I've also added them for the same reason mentioned by Joel. Sometimes I use it because some websites block the job from checking (probably with DDOS protection).
Edit. Actually I think I may have added those tags.
An alternate solution is to perhaps have a config file that uses ignorePatterns for these links.
@mapachurro and @jlwllmr this solution using a config file appears to work: https://github.com/Consensys/doc.zk-evm/pull/476
Let me know if you are happy with me to implement it. This solution is for any Github URL, but we can refine it if required.
Also there are quite a few Twitter URLs that use the escape tab. Should we include https://twitter.com/?
@bgravenorst @mapachurro given we merged the solution, can we close this issue?
Closing as this was fixed a while ago.