Skip to content

Comments

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

Merged
sapphi-red merged 3 commits intorolldown-vitefrom
fix/import-optional-peer-dep-should-throw-an-error
Jun 18, 2025
Merged

fix: importing an optional peer dep should throw an runtime error#167
sapphi-red merged 3 commits intorolldown-vitefrom
fix/import-optional-peer-dep-should-throw-an-error

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