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: changed the dependency @fakerjs/faker from ver. 5.5.3 to 9.3.0 #1393

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

Update: changed the dependency @fakerjs/faker from ver. 5.5.3 to 9.3.0 #1393

wants to merge 2 commits into from
+6,291 −3,918

Conversation

pecpo
Copy link

@pecpo pecpo commented Jan 11, 2025 •
edited
Loading

Closes: #1390

what changed?
upgraded dependency

  1. @fakerjs/faker from ver. 5.5.3 to 9.3.0
  2. Changed several deprecated methods to new ones according to the faker version migration guidelines. (https://fakerjs.dev/guide/upgrading)
  3. Edited one test to be more descriptive of which generator function is failing/deprecated
  4. Passes all test suites

@pecpo
Copy link
Author

pecpo commented Jan 12, 2025

@coditva @appurva21 kindly have a look at this, and suggest any changes if needed

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

Successfully merging this pull request may close these issues.

@faker-js/faker@5.5.3 is deprecated
1 participant
@pecpo