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.

Sign up for GitHub

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

Jump to bottom

refactor: replace uuid dependency for built in nodejs util #404

Merged

vladholubiev merged 1 commit into shelfio:master from tim-hoffmann:master

Dec 1, 2022

Contributor

tim-hoffmann commented Dec 1, 2022

Hi,
I removed the uuid dependency because Node.js has a built in utility for v4 uuids since version 15.
Docs: https://nodejs.org/docs/latest-v16.x/api/crypto.html#cryptorandomuuidoptions


          refactor: replace uuid dependency for built in nodejs util

0c709ff

vladholubiev approved these changes

View reviewed changes

Member

vladholubiev commented Dec 1, 2022

Nice catch! Thanks!

vladholubiev merged commit 7ebaa8b into shelfio:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet