Upgrade nodemon and chokidar
What does this MR do?
@timzallmann started this MR because:
It upgrades the nodemon version which is used by the webpack dev server as we are using a Version from 2018 and this new version for example has
Chokidar upgrade means: massive CPU & RAM consumption improvements. 17x package & deps size reduction.
https://github.com/remy/nodemon/releases
As the main benefit comes from updating chokidar (removing the underlying fsevents dependency), we can force a newer version of chokidar via yarn resolutions. Their public API remained the same, the only reason for the major version bump is because they need a newer node version (>= 8). As we are using these newer node versions already, let's get the benefit from that!
Read more here about the improvements:
- https://paulmillr.com/posts/chokidar-3-save-32tb-of-traffic/
- https://app.renovatebot.com/package-diff?name=chokidar&from=2.1.8&to=3.4.0
Created this follow-up issue for removing the manual overwrite: #219353
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry -
Documentation (if required) -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers -
Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
-
Label as security and @ mention @gitlab-com/gl-security/appsec
-
The MR includes necessary changes to maintain consistency between UI, API, email, or other methods -
Security reports checked/validated by a reviewer from the AppSec team