Skip to content

fix: use python3 in link checker npm scripts#792

Merged
NoahMaizels merged 4 commits intomasterfrom
fix/link-checker-python
Apr 20, 2026
Merged

fix: use python3 in link checker npm scripts#792
NoahMaizels merged 4 commits intomasterfrom
fix/link-checker-python

Conversation

@NoahMaizels
Copy link
Copy Markdown
Collaborator

Summary

Fixed npm scripts to use python3 instead of python since the system only has python3 installed. The check_links.py script already specifies the correct shebang.

Changes

  • Updated package.json check:links and build:check scripts to use python3
  • Created .link-checker-ignore file for managing URLs that shouldn't block CI

Testing

Verified that npm run check:links now executes successfully without python not found errors.

The system has python3 installed but not python. Updated npm scripts
to explicitly use python3, which the check_links.py script already
specifies in its shebang.
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 20, 2026

Deploy Preview for test-twitter-preview-testing-3 ready!

Name Link
🔨 Latest commit 71c3c57
🔍 Latest deploy log https://app.netlify.com/projects/test-twitter-preview-testing-3/deploys/69e5e19cdd2aea00086404fa
😎 Deploy Preview https://deploy-preview-792--test-twitter-preview-testing-3.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 project configuration.

NoahMaizels and others added 2 commits April 20, 2026 15:12
Reverts bad commit reference that was causing build failures.
Keeps python3 fix and .link-checker-ignore file changes.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Resolves merge conflict in .link-checker-ignore by keeping
the real example URL from the dead links PR #793.
@NoahMaizels NoahMaizels merged commit 2de9d3d into master Apr 20, 2026
4 of 9 checks passed
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.

1 participant