Skip to content

fix: importing an optional peer dep should throw an runtime error #167

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

Merged

Conversation

sapphi-red
Copy link
Member

@sapphi-red sapphi-red commented May 12, 2025

Description

See vitejs#20029

The native plugins test fails are expected as this PR only changes the JS plugins.

fixes #165

@sapphi-red sapphi-red force-pushed the fix/import-optional-peer-dep-should-throw-an-error branch from ab5dc4e to 2fdd982 Compare June 17, 2025 03:40
@sapphi-red sapphi-red force-pushed the fix/import-optional-peer-dep-should-throw-an-error branch from 2fdd982 to edfd293 Compare June 17, 2025 03:57
github-merge-queue bot pushed a commit to rolldown/rolldown that referenced this pull request Jun 17, 2025
@sapphi-red sapphi-red force-pushed the fix/import-optional-peer-dep-should-throw-an-error branch from edfd293 to 6a488f3 Compare June 18, 2025 01:20
@sapphi-red sapphi-red marked this pull request as ready for review June 18, 2025 01:20
@sapphi-red sapphi-red force-pushed the fix/import-optional-peer-dep-should-throw-an-error branch from 6a488f3 to 268d892 Compare June 18, 2025 02:31
@sapphi-red
Copy link
Member Author

Native plugins test passed locally with rolldown/rolldown#4995

@sapphi-red sapphi-red merged commit b53667a into rolldown-vite Jun 18, 2025
22 of 28 checks passed
@sapphi-red sapphi-red deleted the fix/import-optional-peer-dep-should-throw-an-error branch June 18, 2025 02:43
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.

motion package breaks with "isValidProp is not a function"
1 participant