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

Emit progress-update event on the listeningApp, allowing update progress to be obtained in the node API. #1666

Closed
wants to merge 3 commits into from
Closed

Conversation

pfdgithub
Copy link

  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

Motivation / Use-Case

Breaking Changes

Additional Info

Just like in the browser, we can now get the compilation progress of the current project in the node API.

new webpackDevServer(webpack, config).listen(port, hostname, function (err) {
  this.on('progress-update', ({ percent, msg }) => {
    console.log(`${percent}% - ${msg}.`);
  });
});

…ess to be obtained in the node API.

```
new webpackDevServer(webpack, config).listen(port, hostname, function (err) {
  this.on('progress-update', ({ percent, msg }) => {
    console.log(`${percent}% - ${msg}.`);
  });
});
```
@jsf-clabot
Copy link

jsf-clabot commented Feb 15, 2019

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Please accept CLA and add tests

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Please accept CLA and add tests

@codecov
Copy link

codecov bot commented Feb 25, 2019

Codecov Report

Merging #1666 into master will decrease coverage by 0.32%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1666      +/-   ##
==========================================
- Coverage   84.39%   84.06%   -0.33%     
==========================================
  Files           7        7              
  Lines         519      521       +2     
  Branches      159      160       +1     
==========================================
  Hits          438      438              
- Misses         64       65       +1     
- Partials       17       18       +1
Impacted Files Coverage Δ
lib/Server.js 79.52% <0%> (-0.42%) ⬇️

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 bf99c26...ee98dc3. Read the comment docs.

@alexander-akait
Copy link
Member

Please accept CLA

@hiroppy
Copy link
Member

hiroppy commented May 30, 2019

Friendly ping @pfdgithub

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.

4 participants