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

Add TypeScript support and Use preset-env #3

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

aminya
Copy link

@aminya aminya commented May 14, 2021

Fixes #2

I added these extras by default. However, we can put them behind a process.env.BABEL_EXTRAS config (or a new set of presets) if you prefer that the default config only use babel-solid-js

Copy link
Author

@aminya aminya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I link this branch to another project it doesn't work. Not sure why. The babel config works when it is inside my babel.config.js, but it doesn't work when used in solid-jest

@maxmilton
Copy link

This package should be focused on solid support in jest rather than a general solution. Let's not bloat it and instead continue to give the consumer control over what additional transforms are performed.

@ryansolid
Copy link
Member

I tend to agree with @maxmilton. I'd love to hear more opinions but I'm inclined to close this one.

@aminya
Copy link
Author

aminya commented Aug 9, 2021

As I said, we can put this behind a config, and move the dependencies to optionalDependencies. If the improvements are composable there is no need to avoid them.

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.

Doesn't work with TypeScript
3 participants