Skip to content

fix: default match rules#40

Merged
thihathit merged 1 commit intomasterfrom
default-matching-rules
May 10, 2025
Merged

fix: default match rules#40
thihathit merged 1 commit intomasterfrom
default-matching-rules

Conversation

@thihathit
Copy link
Copy Markdown
Owner

@thihathit thihathit commented May 6, 2025

Wildcards are defaults to '*' if they're omitted. If that's the correct assumption leave it as they are, so there's less artificial normalizations.

https://developer.mozilla.org/en-US/docs/Web/API/URLPattern/URLPattern#input

@thihathit thihathit force-pushed the default-matching-rules branch from e747450 to 02b7760 Compare May 6, 2025 20:37
@thihathit thihathit marked this pull request as ready for review May 6, 2025 20:43
@thihathit thihathit force-pushed the default-matching-rules branch 2 times, most recently from ad3426f to 38f7857 Compare May 6, 2025 21:17
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented May 6, 2025

Open in StackBlitz

npm i https://pkg.pr.new/thihathit/rutter@40

commit: dbc3a87

@thihathit thihathit force-pushed the default-matching-rules branch from 38f7857 to dbc3a87 Compare May 9, 2025 13:59
@thihathit thihathit requested a review from zwelhtetyan May 9, 2025 14:04
@zwelhtetyan
Copy link
Copy Markdown
Collaborator

LGTM 👀🎉

@thihathit thihathit merged commit beff619 into master May 10, 2025
9 checks passed
@thihathit thihathit deleted the default-matching-rules branch May 10, 2025 10:45
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.3.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants