Skip to content

fix(router-core): params.parse should run as minimally as possible - Alpha #5126

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.

Sign up for GitHub

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

Jump to bottom
Closed
nlynzaad wants to merge 1 commit into from
Closed

fix(router-core): params.parse should run as minimally as possible - Alpha #5126

nlynzaad wants to merge 1 commit into from

Conversation

@nlynzaad
Copy link
Contributor

@nlynzaad nlynzaad commented Sep 11, 2025

merge PR #5124 into alpha

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 11, 2025

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

92 files out of 300 files are above the max files limit of 200.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch alpha-merge-5124

Comment @coderabbitai help to get the list of available commands and usage tips.

@nlynzaad nlynzaad changed the base branch from main to alpha September 11, 2025 15:33
@nlynzaad nlynzaad closed this Sep 11, 2025
@nlynzaad nlynzaad deleted the alpha-merge-5124 branch September 11, 2025 15:33
@nlynzaad nlynzaad restored the alpha-merge-5124 branch September 11, 2025 15:34
@nlynzaad nlynzaad deleted the alpha-merge-5124 branch September 11, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

documentation Everything documentation related package: arktype-adapter package: directive-functions-plugin package: eslint-plugin-router package: history package: react-router package: react-router-devtools package: react-router-ssr-query package: react-start package: react-start-client package: react-start-plugin package: react-start-server package: router-cli package: router-core package: router-devtools package: router-devtools-core package: router-generator package: router-plugin package: router-ssr-query-core package: router-utils package: router-vite-plugin package: server-functions-plugin package: solid-router package: solid-router-devtools package: solid-start package: solid-start-client package: solid-start-plugin package: solid-start-server package: start-client-core package: start-plugin-core package: start-server-core package: start-server-functions-client package: start-server-functions-fetcher package: start-server-functions-server package: start-storage-context package: valibot-adapter package: virtual-file-routes package: zod-adapter

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nlynzaad