-
Notifications
You must be signed in to change notification settings - Fork 156
fix: fixed lang='tsx'
files not working for vue2-jest
#395
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: fixed lang='tsx'
files not working for vue2-jest
#395
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems okay, I am unsure about removing the target: 'es5'
. If CI is okay, it should be fine, though.
@@ -72,7 +72,7 @@ const getTsJestConfig = function getTsJestConfig(config) { | |||
const configSet = new ConfigSet(config.config) | |||
var tsConfig = configSet.typescript || configSet.parsedTsConfig | |||
return { | |||
compilerOptions: { ...tsConfig.options, target: 'es5', module: 'commonjs' } | |||
compilerOptions: { ...tsConfig.options, module: 'commonjs' } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if changing the target: 'es5'
is going to cause any problems. Let's see if CI passes.
I can release over this weekend. Thanks! |
lang='tsx'
files not working for vue2-jestlang='tsx'
files not working for vue2-jest
Forcing an ES5 target isn't ideal for us so I like this change. Do you know when you'll do a release with this in? |
Sorry, I forgot to release this. I will try do this within 48h, tag or ping me if it isn't done by the start of next week. |
Fix lang='tsx' files to be transpiled as tsx.
typescript.transpileModule
does not correctly parse JSX syntax when the file extension is not.tsx
.