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.21k stars 1.72k forks source link

Add node 20 to test matrix #2133

Open BackEndTea opened 11 months ago

netlify[bot] commented 11 months ago

Deploy Preview for nodemon ready!

Name Link
Latest commit 32807b56eae8392111c54102220afccf59a0e1c5
Latest deploy log https://app.netlify.com/sites/nodemon/deploys/650aa8741d2b6b00081e6916
Deploy Preview https://deploy-preview-2133--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 configuration.

remy commented 10 months ago

I don't think the tests are failing because of node 20 - but because something's either wrong in the test or there's a regression somewhere…

I'm trying to have a look but don't have a lot of time (on the computer) atm - but if no one else does, I will do and I will get node 20 in the matrix.

jimmywarting commented 10 months ago

How would you feel about just testing only the latest 3 versions? 16, 18, 20? and setting minium node version to it as well in package.json Could start using newer NodeJS features by doing so.