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: Temporarily remove link checking #214

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

edwarddavidbaker
Copy link
Contributor

Markdown Link Check fails when checking links at https://www.intel.com/content/www/us/ and https://www.intel.com/content/dam/www/. When run from GitHub's virtual machines these links report HTTP 403. A new approach will be necessary to automatically check link validity. Removing automated checks in the mean time to avoid CI failures. The original goal was to automatically monitor Markdown for any broken links and update them as necessary.

Markdown Link Check fails when checking links at
https://www.intel.com/content/www/us/ and
https://www.intel.com/content/dam/www/. When run from GitHub's virtual
machines these links report HTTP 403. A new approach will be necessary
to automatically check link validity. Removing automated checks in the
mean time to avoid CI failures.
@edwarddavidbaker
Copy link
Contributor Author

One future option might be https://github.com/UmbrellaDocs/action-linkspector which is the successor to markdown-link-check. At this point, linkspector fails on files (such as checking PDFs) since it uses Puppeteer.

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