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 source maps for vue2-jest #486

Merged
merged 3 commits into from
Aug 15, 2022
Merged

Fix source maps for vue2-jest #486

merged 3 commits into from
Aug 15, 2022

Conversation

NikhilVerma
Copy link
Contributor

@NikhilVerma NikhilVerma commented Jul 22, 2022

The code has been borrowed from vue3-jest

For me this fixes: #474

@lmiller1990 lmiller1990 self-requested a review August 3, 2022 01:09
@lmiller1990
Copy link
Member

CI is failing - can you take a look?

@NikhilVerma
Copy link
Contributor Author

@lmiller1990 I've fixed the test issues, it was a path mismatch between my machine and the github CI agent. Can you pls re-run the CI? I don't have the permissions to do so.

@john-maddock
Copy link

john-maddock commented Aug 3, 2022

@NikhilVerma @lmiller1990 im using vue2-jest but not using typescript, im also having source map issues. Wondering will this fix it for ?..

@lmiller1990
Copy link
Member

Looks good, I'll leave this open until I've got bandwidth to release, that should be in the next few days. Ping me if it does not happen.

@john-maddock no idea, you could test it out by just copy pasting the changes into your node_modules/vue-jest directory - no need to compile, it's regular JS.

@lmiller1990
Copy link
Member

Let's wait for #483 to merge, then I'll merge this and release the new version.

@lmiller1990
Copy link
Member

Let's just ship this.

@lmiller1990 lmiller1990 merged commit 4530039 into vuejs:master Aug 15, 2022
@lmiller1990
Copy link
Member

It's out https://github.com/vuejs/vue-jest/releases/tag/v28.1.0

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.

Source maps for vue2 SFCs using TS are not created/wrongly created for v28
3 participants