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

Fixes broken links #20

Merged
merged 2 commits into from
Aug 19, 2024
Merged

Fixes broken links #20

merged 2 commits into from
Aug 19, 2024

Conversation

DarikshaAnsari
Copy link
Owner

Copy link

netlify bot commented Aug 11, 2024

Deploy Preview for in-toto-docs ready!

Name Link
🔨 Latest commit ec0ad9c
🔍 Latest deploy log https://app.netlify.com/sites/in-toto-docs/deploys/66bb795f5c48bd000862ff3b
😎 Deploy Preview https://deploy-preview-20--in-toto-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@DarikshaAnsari
Copy link
Owner Author

cc @lukpueh

Copy link
Collaborator

@lukpueh lukpueh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you run @chalin's link checker command from #19? I still get an error for one target:

npm run check:links

...

> make check-links

tmp/bin/htmltest --skip-external
Skipping the checking of external links.
htmltest started at 01:58:06 on public
========================================================================
index.html
  target does not exist --- index.html --> /docs/get-started/
========================================================================
✘✘✘ failed in 9.08325ms
1 errors in 20 documents
make: *** [check-links] Error 1

@DarikshaAnsari DarikshaAnsari requested a review from lukpueh August 13, 2024 15:19
@DarikshaAnsari DarikshaAnsari merged commit 6cc61ba into main Aug 19, 2024
5 checks passed
@chalin chalin deleted the patch-broken-links branch August 22, 2024 19:09
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.

Fix broken links
2 participants