Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

fix: rome.tools broken link #29

Closed
wants to merge 1 commit into from

Conversation

rewrite0w0
Copy link
Contributor

This is a fix related to #22. incorporated @suica's wise comments.

@rewrite0w0 rewrite0w0 closed this Nov 14, 2023
@suica
Copy link
Contributor

suica commented Nov 14, 2023

Maybe we can simply add https://rome.tools/ to ignored domains, and somehow find a way to fix links to other doc pages for errors like this

* [ERR] [file:///home/runner/work/javascript-parser-in-rust/javascript-parser-in-rust/i18n/ja/docusaurus-plugin-content-docs/current/overview](file:///home/runner/work/javascript-parser-in-rust/javascript-parser-in-rust/i18n/ja/docusaurus-plugin-content-docs/current/overview) | Failed: Cannot find file

@mysteryven
Copy link
Contributor

mysteryven commented Nov 14, 2023

Seems the relative missing doc can be solved by this 🤔

-b, --base <BASE>
          Base URL or website root directory to check relative URLs e.g. https://example.com or `/path/to/public`

@mysteryven
Copy link
Contributor

Feel free to reopen this, one link catch is also welcome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants