Skip to content
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

Ship regexparam with optional wildcards #375

Merged
merged 4 commits into from
Nov 2, 2023
Merged

Conversation

molefrog
Copy link
Owner

@molefrog molefrog commented Nov 1, 2023

Adds src/regexparam.js which was extracted from https://github.com/lukeed/regexparam
It is a temporary fix until lukeed/regexparam#25 is shipped

Copy link

codesandbox bot commented Nov 1, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Copy link

github-actions bot commented Nov 1, 2023

size-limit report 📦

Path Size
packages/wouter/esm/index.js 2.07 KB (+0.19% 🔺)
packages/wouter/esm/use-browser-location.js 693 B (0%)
packages/wouter/esm/memory-location.js 836 B (0%)
packages/wouter/esm/use-hash-location.js 709 B (0%)
packages/wouter-preact/esm/index.js 1.95 KB (+0.51% 🔺)
packages/wouter-preact/esm/use-browser-location.js 590 B (0%)
packages/wouter-preact/esm/use-hash-location.js 590 B (0%)
packages/wouter-preact/esm/memory-location.js 703 B (0%)

@molefrog
Copy link
Owner Author

molefrog commented Nov 1, 2023

@jeetiss I've decided not to create a workspace package, since it makes it harder to maintain builds and ship types. The code itself is 30 LOC, so I guess it is fine for the time being. I really hope that eventually it lands there, so we could remove the fix.

@molefrog molefrog merged commit f5441c6 into v3 Nov 2, 2023
2 checks passed
@molefrog molefrog deleted the feat/optional-wildcards branch November 2, 2023 12:21
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