search

remy / nodemon

Monitor for any changes in your node.js application and automatically restart the server - perfect for development
http://nodemon.io/
MIT License
26.22k stars 1.72k forks source link

core(deps): remove update notifier #2015

Closed jimmywarting closed 2 years ago

jimmywarting commented 2 years ago

This will remove 76 sup dependencies some of us do not want to have to download and risk getting any security issues along with it. This will drastically lower the time/bandwidth to install the nodemon package

the way to notify user about outdated packages should be done via npm deprecate instead to let user know that they are installing a outdated version instead

fix #1961 fix #1995

netlify[bot] commented 2 years ago

Deploy Preview for nodemon ready!

Name Link
Latest commit 47e8d1c35cff1844b1c439e593e2661ac49e191e
Latest deploy log https://app.netlify.com/sites/nodemon/deploys/6298db13e6f3a9000911b01a
Deploy Preview https://deploy-preview-2015--nodemon.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

remy commented 2 years ago

As discussed in tickets, flat out removing the update logic isn't a path I intend to take. I'm keen to remove update-notifier but whilst replacing it with nodemon's own simpler system.