nfroidure / svgicons2svgfont

Concatenate SVG icons and ouput an SVG font
http://nfroidure.github.io/svgiconfont/
MIT License
338 stars 71 forks source link

Dependency on deprecated versions of glob and inflight (with a scary message for the latter) #169

Closed anomiex closed 3 months ago

anomiex commented 5 months ago

Issue

I'm a gentledev I:

Expected behavior

No deprecation warnings on npm add svgicons2svgfont.

Actual behavior

$ npm add svgicons2svgfont
npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated glob@8.1.0: Glob versions prior to v9 are no longer supported

added 13 packages in 2s

Steps to reproduce the behavior

Run npm add svgicons2svgfont in an empty project.

Debugging informations

Other notes

It looks like updating glob would be straightforward here, the breaking changes in v9 and v10 don't seem to affect the way this project uses glob. The one code change you might consider is whether you want to sort the results of glob.sync() since v9+ no longer does any sorting itself.