webfonts-generator icon indicating copy to clipboard operation
webfonts-generator copied to clipboard

Significantly Outdated Dependency and Sub-Dependency

Open bdjnk opened this issue 7 years ago • 3 comments

The current "version": "0.4.0" of webfonts-generator includes "svgicons2svgfont": "^5.0.0", whose latest minor version is six months old. They're currently at "version": "8.0.0".

Worse still, "svgicons2svgfont": "^5.0.0" appears to be including "svg-pathdata": "^1.0.4", which is a year old. They're currently at "version": "4.0.1".

All this is important to me because bugs have cropped up for me in svg-pathdata while using icon-font-generator (which uses webfonts-generator) and I can't tell if the problems might be resolved simply by being up to date.

bdjnk avatar Jan 03 '18 14:01 bdjnk

I forked the repo and updated the dependencies, then ran the tests. The tests passed, except for a small issue with the RENDERED_TEMPLATE having a return character \r before the newline. I updated the test too so that it passed. The only other update I had to make was calling svgicons2svgfont with new.

Mind testing it out? Should anything else be done before submitting a PR?

https://github.com/markadrake/webfonts-generator/commit/387029ee051a181a1740177ca5eea7d64ee58fe2

markadrake avatar Jan 11 '18 19:01 markadrake

I've tested your fork on my set of SVGs. No errors presented and the bugs I was experiencing have evidently been resolved.

Thank you @markadrake!

bdjnk avatar Jan 12 '18 18:01 bdjnk

You are very welcome @bdjnk. It also resolved some quirks I noticed with my own SVG icons.

Wondering how to push the PR forward though - not a lot of experience on my end committing to public open source projects like this. Any tips or suggestions? Looks like part of the reason my PR failed was due to some outdated node versions so that's immediately a major release. I get that, but no response from the maintainer :/

markadrake avatar Jan 26 '18 15:01 markadrake