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

fix(sveltekit): Bump magicast to handle vite configs declared as variables #380

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented Jul 27, 2023

Previously, magicast didn't handle adding a vite plugin into a vite config object that was separately declared as a variable instead of being directly exported as default:

const config = {};
export default config;

Since unjs/magicast#69 was just merged 🎉 and released, let's bump magicast to the latest version to fix this.

closes #362

@Lms24 Lms24 requested review from lforst and AbhiPrasad July 27, 2023 10:25
@Lms24 Lms24 self-assigned this Jul 27, 2023
@Lms24 Lms24 merged commit d487995 into master Jul 27, 2023
@Lms24 Lms24 deleted the lms/fix-sveltekit-bump-magicast-variable-declarations branch July 27, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants