-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
document path-to-regexp change in express 5 migration guide #1114
Conversation
Pre-req: pillarjs/router#42 |
one approval, and this is should be ready to go in! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Potentially we could also add explanations for the different suffixes?
@LinusU - from your approval, I am assuming that this comment is not a blocker? @expressjs/express-tc - this is all set to land, towards express 5 GA. Is there anyone with spare bandwidth to review and merge this? |
Correct 👍 Although @jonchurch have now provided an extend description which I think we should use 👏 |
I believe we need to wait for the express release with this change to be published prior to landing on the site (i.e. 5.0.0-beta.1) |
2a21f19
to
00062c1
Compare
bb4652e
to
726bf05
Compare
da3de79
to
4cea036
Compare
No description provided.