Skip to content

Conversation

@calebegg
Copy link

path-to-regexp does not support this syntax at all. There was a PR to implement it, but it seems pretty dead. pillarjs/path-to-regexp#362

path-to-regexp does not support this syntax at all. There was a PR to implement it, but it seems pretty dead. pillarjs/path-to-regexp#362
@calebegg calebegg requested a review from a team as a code owner October 10, 2025 17:04
@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit bbf38d7
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/68e93c8752d0300008171631
😎 Deploy Preview https://deploy-preview-2080--expressjscom-preview.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.

Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution, but unfortunately we maintain the documentation for both Express v4 and v5, so we can't remove this information that easily.

It may be confusing, and perhaps we can improve the way we present this information.

@calebegg
Copy link
Author

I see. Maintaining one source of documentation for two dramatically different versions of an application seems challenging. I would suggest that y'all take a broader approach to that issue than can reasonably be addressed in one PR.

@calebegg calebegg closed this Nov 15, 2025
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