Skip to content

don't ignore GitHub links and update broken ones - #2483

Draft
krzysdz wants to merge 1 commit into
expressjs:mainfrom
krzysdz:fix-github-URLs
Draft

don't ignore GitHub links and update broken ones#2483
krzysdz wants to merge 1 commit into
expressjs:mainfrom
krzysdz:fix-github-URLs

Conversation

@krzysdz

@krzysdz krzysdz commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Lychee apparently can avoid GitHub rate limits if it has access to the GITHUB_TOKEN. It has problems with some fragments (see lycheeverse/lychee#2128), but this functionality is not enabled in CI and I couldn't make it work without #2356.

Related PRs to external docs

Waiting for update external docs action.

Renamed accounts

Some people mentioned in one of the blog posts have renamed their accounts and old links return 404. I managed to find their current account names and some evidence that those are the correct accounts.

old current details
attrid riddlew expressjs/express#5195, 4.18.3 release notes (2024) use the name riddlew, IA has captures of attrid only in April-August 2025 and riddlew until January 2025 and since October 2025 with a 404 in May 2025; must have been a temporary name change
axhuwastaken nondeterministic-polynomial #1719 (comment)
bigbigDreamer iambinlin jshttp/fresh#38 (review)
GeorgeShvab heorhiiss #1680 (comment), #1703 (comment), #1682
hamirmahal hirehamir "@hamirmahal made their first contribution in fix: usage of Invalid action input 'persist-credentials' for actions/setup-node@v4 in ci.yml #6256" (expressjs/express#6316)
MaoShizhong mao-sz #1515 (review)

@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit ea07e47
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6a727d46a7b3bd00082bfe5d
😎 Deploy Preview https://deploy-preview-2483--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 98 (🟢 up 1 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@krzysdz krzysdz added docs Issues/pr concerning content github_actions Pull requests that update GitHub Actions code labels Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Issues/pr concerning content github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant