Skip to content

Upgrade nodemon and chokidar

Tim Zallmann requested to merge tz-upgrade-nodemon into master

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:

Created this follow-up issue for removing the manual overwrite: #219353

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

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
Edited by Lukas 'ai-pi' Eipert

Merge request reports