Skip to content
GitLab
    • Explore Projects Groups Snippets
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C create-react-app
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,547
    • Issues 1,547
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 417
    • Merge requests 417
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • create-react-app
  • Merge requests
  • !1377

webpack-dev-server patch for 'still-ok' success status

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/robbiedigital/hotfix/still-ok-patch into master 8 years ago
  • Overview 6
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: robbiedigital

handleSuccess function in react-dev-utils/webpackHotDevClient.js does not run for message.type of 'still-ok'. added 'still-ok' to the switch statement so in can reload the window and appropriately clear error messages.

Loading
Loading

Activity


  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: facebook-github-bot

    Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

    If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions.

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: facebook-github-bot

    Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: gaearon

    Could you describe how to test this? What is the workflow, and what is the result before / after?

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: robbiedigital

    I was writing a plugin that emitted errors to webpack on fail. However on success the compilation.errors object is not touched (left empty) so webpack emits a 'still-ok' status. Which webpackHotDevClient does not know how to handle.

    You can see the code for 'still-ok' status at https://github.com/webpack/webpack-dev-server/blob/master/lib/Server.js#L467

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Merged by: gaearon at 2017-01-12 18:27:59 UTC

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: gaearon

    Seems good, thanks.

Please register or sign in to reply
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
0
None
0
None
    Assign labels
  • Manage project labels

Milestone
MathJax v2.7.5
MathJax v2.7.5
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
0
0 participants
Reference: mathjax/MathJax!2006
Source branch: github/fork/robbiedigital/hotfix/still-ok-patch

Menu

Explore Projects Groups Snippets