faker icon indicating copy to clipboard operation
faker copied to clipboard

Published 20 hours ago • verified publisher icon faker-js

Move random.word to word.any

Open pkuczynski opened this issue 2 years ago • 7 comments

Clear and concise description of the problem

Most of the methods of random already found their new home, like random.number > datatype.number

Suggested solution

Move random.word to word.any and deprecate random.word.

Alternative

No response

Additional context

https://github.com/faker-js/faker/pull/276#issuecomment-1024251316

pkuczynski avatar Jan 28 '22 20:01 pkuczynski

I'll take this! Simple enough.

andrewgremlich avatar Jan 29 '22 22:01 andrewgremlich

I see this PR though... should I wait until that is merged? https://github.com/faker-js/faker/pull/276

andrewgremlich avatar Jan 29 '22 23:01 andrewgremlich

I see this PR though... should I wait until that is merged? https://github.com/faker-js/faker/pull/276

In principle no changes like this will be merged until 6.0 is released. So I would wait with this, yes.

pkuczynski avatar Jan 30 '22 00:01 pkuczynski

Since 6.0 was released, I'll continue with this one!

andrewgremlich avatar Mar 28 '22 15:03 andrewgremlich

Since 6.0 was released, I'll continue with this one!

Please don't. It has been already discussed and I am on it.

pkuczynski avatar Mar 28 '22 15:03 pkuczynski

Sounds good!

andrewgremlich avatar Mar 28 '22 15:03 andrewgremlich

I would like to get this implemented.

ST-DDT avatar Aug 28 '22 06:08 ST-DDT