The sprintf package is deprecated in favor of sprintf-js

jeresig / i18n-node-2

Lightweight simple translation module for node.js / express.js with dynamic json storage. Uses common __('...') syntax in app and templates.

MIT License

507 stars 79 forks source link

The sprintf package is deprecated in favor of sprintf-js #120

Closed hbakhtiyor closed 5 years ago

avallete commented 5 years ago

The title is pretty explicit.

Did we have any update on this issue ?

PS: I mention as an additional intel that the sprintf-js package does not seem to make any breaking change (I've checked all the releases and searched for 'migration guides'). So the update should be straight forward.

Plus, a PR is already waiting to be merged to address this issue: https://github.com/jeresig/i18n-node-2/pull/122

@gjuchault could you please check and merge this PR ?

gjuchault commented 5 years ago

@avallete I just saw the PR and merged it thanks :)