Skip to content
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

ci(link-check): check docs dead link #17506

Closed
wants to merge 6 commits into from
Closed

Conversation

baicie
Copy link
Contributor

@baicie baicie commented Jun 18, 2024

Description

check docs dead http/https link

Copy link

stackblitz bot commented Jun 18, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@patak-dev
Copy link
Member

Thanks for the PR @baicie! We discussed this with @bluwy and @brc-dd and we think that it is better to avoid checking every link on each commit. Doing all these requests seems a bit wasteful given that we will be checking them constantly. There are services like https://www.screamingfrog.co.uk/seo-spider/ that could be used. Maybe there are some free alternatives for OSS projects. For now, I think we can't just rely on Vite users and contributors. The docs has a lot of traffic and as soon as there are is a broken link, we get a quick fix from the community.

@patak-dev patak-dev closed this Jul 1, 2024
@baicie baicie deleted the feat/check-links branch July 1, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants