search

raszi / node-tmp

Temporary file and directory creator for node.js
MIT License
736 stars 92 forks source link

Remove template option #239

Closed silkentrance closed 2 years ago

silkentrance commented 4 years ago

The template option in its current form is a useless and utterly redundant feature and must be eliminated with no further warning as to its deprecation.

One can always use the standard prefix/postfix/dir option to achieve a similar result, with the random part being 12 instead of 6 chars.

We could also add a name length option that will allow the user to specify name lengths in the range of 6..18 characters.

raszi commented 4 years ago

The template feature was added because tmp was based on mkstemp.

It might seem redundant but before sunsetting it I believe we should do some analysis of its usage.

silkentrance commented 4 years ago

@raszi how would you make such a usage analysis? integrate google analytics? 😬 - the most part of the users of tmp do not actually care unless we break the API.