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(babel): ignore babelrc for preeval and shaker transforms #1313

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

Anber
Copy link
Collaborator

@Anber Anber commented Aug 2, 2023

Motivation

Some Babel plugins generate code that the evaluator can not handle (see #1308).

Summary

An assumption from #1297 was wrong. Even though we have to resolve babelrc to parse files, we can't use resolved config for pre-eval transformations since it can generate a lot of unprocessable code. The solution is to split the config into parse and transform parts with only the necessary plugins in transform.

@changeset-bot
Copy link

changeset-bot bot commented Aug 2, 2023

🦋 Changeset detected

Latest commit: 13f58be

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 23 packages
Name Type
@linaria/babel-preset Minor
@linaria/shaker Minor
@linaria/testkit Minor
@linaria/utils Minor
@linaria/cli Patch
@linaria/esbuild Patch
linaria Patch
@linaria/rollup Patch
@linaria/stylelint Patch
@linaria/vite Patch
@linaria/webpack4-loader Patch
@linaria/webpack5-loader Patch
linaria-website Patch
vite-example Patch
@linaria/atomic Patch
@linaria/core Patch
@linaria/griffel Patch
@linaria/react Patch
@linaria/tags Patch
esbuild-example Patch
rollup-example Patch
vpssr-linaria-solid-example Patch
@linaria/webpack-loader Patch

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

@Anber Anber merged commit ae162f4 into master Aug 2, 2023
5 checks passed
@Anber Anber deleted the issue/1308 branch August 2, 2023 13:30
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.

1 participant