Skip to content

docs: fix typo 'middlware' -> 'middleware' in serve-index - #2449

Closed
monteirodemeloluca-ux wants to merge 1 commit into
expressjs:mainfrom
monteirodemeloluca-ux:fix/typo-middleware-serve-index
Closed

docs: fix typo 'middlware' -> 'middleware' in serve-index#2449
monteirodemeloluca-ux wants to merge 1 commit into
expressjs:mainfrom
monteirodemeloluca-ux:fix/typo-middleware-serve-index

Conversation

@monteirodemeloluca-ux

Copy link
Copy Markdown

Summary

  • Fixes a typo in the English serve-index middleware doc: "middlware" -> "middleware".

Test plan

  • Docs-only change, no build/test impact.

@monteirodemeloluca-ux
monteirodemeloluca-ux requested a review from a team as a code owner July 14, 2026 13:57
@netlify

netlify Bot commented Jul 14, 2026

Copy link
Copy Markdown

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit 9aa7fee
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6a564037b5090c000863245f
😎 Deploy Preview https://deploy-preview-2449--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 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

This file is fetched from README of serve-index and the typo should be fixed there. There are 3 other (closed) PRs here (#2280, #2295 and #2299), but nobody bothered to submit one to the right repo.

I have added it to expressjs/serve-index#164 together with other fixes to that README.

@krzysdz krzysdz closed this Jul 14, 2026
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.

2 participants