Skip to content
This repository has been archived by the owner on Sep 3, 2022. It is now read-only.

chore(deps): update uuid to v8 since v3.4 is deprecated #272

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cajotafer
Copy link

@cajotafer cajotafer commented May 27, 2022

Description

I am getting this warning using Yarn 2

uuid@npm:3.4.0 is deprecated: 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.

I realized this package is using [email protected] and I think is a good idea to upgrade it to v8, no breaking changes included.
I have followed this doc https://github.com/uuidjs/uuid#upgrading-from-uuid3

Test plan

  • Testing completed successfully using - tried to run yarn test but I've got an error IDK exactly how to fix.
    image

Release plan

HISTORY.md has been updated.

Checklist

  • Thorough explanation of the issue/solution, and a link to the related issue
  • CI tests are passing
  • Unit tests were written for any new code
  • Code coverage is at least maintained, or increased.

Resolves #273

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

Successfully merging this pull request may close these issues.

Udgrade [email protected] to v7 or v8 since it's deprecated
1 participant