Extend Docs to Describe Expected Usage with TypeScript

[xDivisionByZerox]

Note

The updated requirements for the issues can be found in this comment.

Pre-Checks

• Follow our Code of Conduct.
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Faker issue and not related to a combination with another package.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.
• I am willing to provide a PR.

Describe the bug

In a project where TypeScript is not configured to run in strict mode, the function faker.helpers.arrayElement can be called with null and undefined, which leads to errors during runtime.

Minimal reproduction code

https://stackblitz.com/edit/faker-js-demo-c67mtj?file=index.ts

Additional Context

This might be true for ALL functions, as null is assignable to everything in non-strict mode.

• This configuration was not discussed in Discuss boundaries of type-less JavaScript support #2725 (comment).

Environment Info

-

Which module system do you use?

• CJS
• ESM

Used Package Manager

pnpm

[ST-DDT]

IMO we clearly define our needs/requirements.
If you choose to ignore them using js, non strict ts or unsafe casts, then you are on your own.

[xDivisionByZerox]

IMO we clearly define our needs/requirements. If you choose to ignore them using js, non strict ts or unsafe casts, then you are on your own.

I don't know if I agree with this statement, especially since TypeScripts default is still non-strict mode.

[ST-DDT]

Team Decision

• We assume that the user either uses strict TypeScript or handles the issues arising from not using it themselves.

[AmaanRS]

Can you please explain to me what exactly do i have to do, like do i have to types for this ?

[xDivisionByZerox]

@AmaanRS I'm not sure if I understand correctly. Are you asking for the tasks to do to work on the issue? 🤔

[ST-DDT]

Can you please explain to me what exactly do i have to do, like do i have to types for this ?

@AmaanRS This is a documentation task, so it doesn't need any code changes.

---

@xDivisionByZerox I'm not exactly sure what should be done for this issue either.
Could you please explain what documentation do you want where?

[xDivisionByZerox]

@xDivisionByZerox I'm not exactly sure what should be done for this issue either.
Could you please explain what documentation do you want where?

Of couse. IMO the TypeScript section of our docs should add a note box hinting the user that using faker without (TS) strict mode might lead to unexpected behaviour which will not be worked around from our side.