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.

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

Update deprecated uuid #1417

Open
abdulrahman-khankan opened this issue Jul 8, 2021 · 2 comments · May be fixed by #1413
Open

Update deprecated uuid #1417

abdulrahman-khankan opened this issue Jul 8, 2021 · 2 comments · May be fixed by #1413

Comments

@abdulrahman-khankan
Copy link

nyc uses a deprecated version of uuid which triggers npm warnings:

npm WARN deprecated uuid@3.4.0: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.

Could you please update to v8?

Older related issues from March 2020:
@cschanaj cschanaj linked a pull request Jul 18, 2021 that will close this issue
Open
@mgordel mgordel mentioned this issue Apr 21, 2022
Closed
@AncientSwordRage
Copy link

Are we ok to deprecate support for node 10 to make this pass?

@ericcornelissen
Copy link

ericcornelissen commented Jul 12, 2023 •
edited

It seems this has resolved itself with the release of istanbul-lib-processinfo@2.0.3 last year. Installing nyc@15.0.1 or later today in a fresh project results in (only) uuid@8.3.2 being installed. Note that nyc@15.0.0 still has a direct dependency on uuid@^3.3.3. I did not test anything pre-v15.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(deps-dev): bump deprecate package uuid@3.x cschanaj/nyc
3 participants
@ericcornelissen @AncientSwordRage @abdulrahman-khankan