Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace sprintf dependency with built-in or sprintf-js #69

Open
HamletDRC opened this issue Nov 24, 2014 · 0 comments
Open

Replace sprintf dependency with built-in or sprintf-js #69

HamletDRC opened this issue Nov 24, 2014 · 0 comments

Comments

@HamletDRC
Copy link

The owner of sprintf (https://github.com/maritz/node-sprintf) says that node-sprintf is no longer maintained. He suggests replacing your sprintf with either the built-in Javascript methods or use the sprintf-js package.

My issue with Whiskey's usage of sprintf is that its package.json file does not contain a license declaration. This is a problem for my legal department. I asked the maintainer to update the file as a separate issue, but it is unclear if he'll do it. He recommended that I create an issue with Whiskey asking the project to remove the dependency. Discussion here: maritz/node-sprintf#18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant