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

Remove unused faker from the dependencies #233

Open
rolfl opened this issue Jun 14, 2021 · 3 comments
Open

Remove unused faker from the dependencies #233

rolfl opened this issue Jun 14, 2021 · 3 comments

Comments

@rolfl
Copy link

rolfl commented Jun 14, 2021

faker is not used in the code base.

$ npx depcheck
Unused dependencies
* faker

This unused dependency simply adds bloat.

@xDivisionByZerox
Copy link

xDivisionByZerox commented Jul 15, 2022

Can any maintainer confirm this (just in case)? I would appreciate this getting attention due to faker not being maintained anymore, but still gaining a lot of traffic.

I'm willing to provide a PR to fix this.

@whitlockjc
Copy link
Member

I've not looked at the code in a while but we use faker indirectly via json-schema-faker:

var jsf = require('json-schema-faker');
to allow for Parameter#getSample to generate an example for a Parameter's schema. If faker is no longer maintained, either we need to find a replacement or we need to remove the ability to register custom format generators and likely sample generation.

@xDivisionByZerox
Copy link

xDivisionByZerox commented Jul 15, 2022

I'm a team member from the faker-js/faker library which is considered the official fork from the old faker package. So I can assure you, other team members are already working on migrating the faker package in json-schema-faker (see here).

So your lib really just needs to remove the faker package from your dependencies in the package.json file.

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

3 participants