-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
ES6 refactoring in DependenciesBlock and Module #4632
ES6 refactoring in DependenciesBlock and Module #4632
Conversation
I did not add/change any code logic, but @webpack-bot tells me |
@smelukov This message is based on Code Coverage :) ( |
It looks like this Pull Request doesn't include enough test cases. @smelukov Please add more test cases. See test readme for details how to write test cases. |
The minimum test ratio has been reached. Thanks! |
The most important CI builds succeeded. Great work so far. |
The Code Coverage for the code you refactored was low before, so if you touch it codecov reports also low coverage for your PR. For the ES6 refactoring PRs that's fine for me. But thanks for adding test cases 😄. Bot seem to have a positive effect 🎉. |
Thanks |
What kind of change does this PR introduce?
Refactoring of Webpack Core
Did you add tests for your changes?
Yes
Summary
More ES6 code in Webpack Core (
DependenciesBlock
andModule
)Inspired #4099
Does this PR introduce a breaking change?
No