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

[BUG FIX] Use fixed version of react-dev-utils #3959

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

igor-dv
Copy link
Member

@igor-dv igor-dv commented Aug 5, 2018

Issue: #3953 #3956

What I did

Somehow, having a caret in "react-dev-utils": "^6.0.0-next.3e165448" is resolved to an older version of react-dev-utils - react-dev-utils@6.0.0-next.b2fd8db8, which is not compatible with webpack 4.

It worked in our examples because it locked with the right version in the yarn.lock, but a fresh install might change this.

@codecov
Copy link

codecov bot commented Aug 5, 2018

Codecov Report

Merging #3959 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3959   +/-   ##
=======================================
  Coverage   39.62%   39.62%           
=======================================
  Files         431      431           
  Lines        5443     5443           
  Branches      740      740           
=======================================
  Hits         2157     2157           
  Misses       2903     2903           
  Partials      383      383

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44b36a7...d62e4af. Read the comment docs.

@igor-dv igor-dv changed the title Use fixed version of react-dev-utils [BUG FIX] Use fixed version of react-dev-utils Aug 5, 2018
@pksunkara
Copy link
Member

Yup, a fresh install in vue-kitchen-sink is encountering the same issue. 4.0.0-alpha.15 is completely non-working because of this. I would recommend merging this and publish a new alpha.

@igor-dv
Copy link
Member Author

igor-dv commented Aug 5, 2018

CC. @shilman

@ndelangen ndelangen merged commit 89df73e into master Aug 6, 2018
@ndelangen ndelangen deleted the react-dev-utils-fixed-versopn branch August 6, 2018 08:12
@ndelangen
Copy link
Member

@shilman please release this asap if you can.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants