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

update webpack-dev-middleware dependency to v3.1.3 #1392

Merged
merged 2 commits into from
May 2, 2018

Conversation

dennispg
Copy link
Contributor

  • This is a bugfix
  • This is a code refactor
  • This is a test update
  • This is a typo fix
  • This is a metadata update

Motivation / Use-Case

Spaces in the outputPath are not being handled correctly on Windows.
This change addresses the regression which introduced the bug and is being tracked in #1375

The only change made is that the package.json has been updated to reference v3.1.3 of webpack-dev-middleware.

The issue was corrected here:
webpack/webpack-dev-middleware#297

@jsf-clabot
Copy link

jsf-clabot commented Apr 27, 2018

CLA assistant check
All committers have signed the CLA.

package.json Outdated
@@ -67,7 +67,7 @@
"spdy": "^3.4.1",
"strip-ansi": "^3.0.0",
"supports-color": "^5.1.0",
"webpack-dev-middleware": "3.1.2",
"webpack-dev-middleware": "^3.1.3",
Copy link
Member

@alexander-akait alexander-akait Apr 27, 2018

Choose a reason for hiding this comment

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

Thanks for PR, please don't change semver, it is can break build in future. I.E. remove ^

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yikes, didn't even realize I did that! sorry, corrected.

@codecov
Copy link

codecov bot commented Apr 27, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1392   +/-   ##
=======================================
  Coverage   79.14%   79.14%           
=======================================
  Files           6        6           
  Lines         494      494           
  Branches      160      160           
=======================================
  Hits          391      391           
  Misses        103      103

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 69a90c7...f692948. Read the comment docs.

@alexander-akait
Copy link
Member

/cc @SpaceK33z

@dennispg
Copy link
Contributor Author

dennispg commented May 2, 2018

hi @SpaceK33z any chance this can be merged?

@SpaceK33z SpaceK33z merged commit 623d337 into webpack:master May 2, 2018
@SpaceK33z
Copy link
Member

Sorry for the delay.

@dennispg
Copy link
Contributor Author

dennispg commented May 2, 2018

cheers! no worries at all, thank you!

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.

4 participants