-
Notifications
You must be signed in to change notification settings - Fork 312
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
Editorial: Update link texts upon updates in the URLPattern specification #1707
Conversation
Updates in the URLPattern specification was requested during the w3c#1701 review. The issues for the request has been resolved: - whatwg/urlpattern#217 - whatwg/urlpattern#218 However, during the specification updates, the algorithm names are also updated, and adjustment is needed.
@mkruisselbrink This simply fixes the broken links to the URLPattern specification. Will you take a look? Thank you. |
I guess I should add prefix "Editorial:" for this kind of changes. |
@domenic Can I ask you to take a look? |
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 with nit
Co-authored-by: Domenic Denicola <[email protected]>
@domenic Thank you for the review. |
Updates in the URLPattern specification was requested during the #1701 review.
The issues for the request has been resolved:
However, during the specification updates, the algorithm names are also updated and arguments are changed.
This is adjustments for them.
Preview | Diff