Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign up[BREAKING CHANGE] remove inline source-map on process() output #255
Conversation
Cool, but won't this be a breaking change for anyone on an old version of jest? We need a good strategy to handle this. I wonder if we can detect the version of jest somehow (maybe by importing package.json?) Any ideas? |
This package specifies
|
Memo: |
@lmiller1990 This version is still in beta. |
a5841fc
to
8b220c4
Ok let's do it. Most people should be easily able to move to Jest 25. Can you write a short "migration guide" explaining the breaking change and what people might need to do to upgrade so we can include that in the release notes? For now I will merge this, but if you can include the upgrade notes here (just as a comment) I can include that with the release. I will do a release in the next few days. |
What about this one? Breaking ChangeIn this version, # npm
> npm install jest@^25.5.0
# yarn
> yarn add jest@^25.5.0 |
Hi there. Thanks for the fix! Btw, I'm confused about having Is there any information about the future of both versions? |
This was merged into 4.0.0-beta. Maybe we need to do a release? I am not sure if we did once since this was merged. Can you check? Basically: v4 = Vue 2 I can make the next release rc.1 for v4... the release goal isn't super clear, but seems many people are using it fine in production. |
I see that the latest 4-beta was released on |
Is it possible to get this fix in v5 too please? |
I released a new 4.0.0 today. So it should be fixed there. This change Is someone able to try porting this to 5.0.0? It should be quite easy. I have many other things to do in the meantime for Vue 3 release. Otherwise I can do it next week. If someone can do it sooner I can do a release just for this. Btw, I am exploring some ideas for a |
Resolve #241 and facebook/jest#10089
In
jest@25.5.0
or later, coverage reporter creates an inline source-map frommap
property. (facebook/jest#9811)But
vue-jest
already adds inline source-map with process().So reporter throws error due to duplicate source maps.
This PR avoids duplicates by removing
map propertyinline source map.