-
Notifications
You must be signed in to change notification settings - Fork 367
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
fix: fix remixRouter not existed in conventional routes #4240
Conversation
🦋 Changeset detectedLatest commit: 0cedb95 The changes in this PR will be included in the next version bump. This PR includes changesets to release 214 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4240 +/- ##
==========================================
- Coverage 57.59% 56.33% -1.26%
==========================================
Files 672 662 -10
Lines 17794 18005 +211
Branches 3876 3936 +60
==========================================
- Hits 10249 10144 -105
- Misses 6934 7200 +266
- Partials 611 661 +50 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at 3b46c08
This pull request fixes a bug in the
@modern-js/runtime
package that causes an error when using conventional routes instead of remix routes. It also adds a changeset file to document the patch updates and the fix messages for the changelog.Details
🤖 Generated by Copilot at 3b46c08
remixRouter
inrouterPlugin
function (link)@modern-js/runtime
package with fix messages in English and Chinese (link)Related Issue
Checklist
pnpm run change
.