avoidwork/filesize.js

Not compatible with Node.js v12?

medikoo opened this issue · 3 comments

medikoo commented

Recently following the CHANGELOG, I've upgraded filesize from v8 to v10.

There are no notes in CHANGELOG that support for Node.js v12 was dropped. Yet suddenly, I got a report from a user stating that you log a warning that Node.js v12 is not supported. Is that the case?

If it's the case, can you indicate that in the CHANGELOG file to avoid further confusion (?) Otherwise, can you update the warning notice so it doesn't scream at Node.js v12 users

avoidwork commented

The lib doesn't emit such a message, but I can lower the engine to 10.0.0 because it has bigint support.

The changelog is generated, it's hard to guess what would matter to a person in the future when considering my writing style.

medikoo commented

@avoidwork npm emits that message based on metadata in package.json. Also, you've already fixed that once: #161 :)

avoidwork commented

@medikoo this is fixed with 10.0.6, such that 10.4.0 is the minimum version.

Regarding min version... I moved it from 0.4.0 to current stable after 11 years, it's interesting to see what products are using what versions due to RTE, etc..

  • 🎉1