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

fix sentence generator #10

Merged
merged 1 commit into from
Dec 7, 2022
Merged

fix sentence generator #10

merged 1 commit into from
Dec 7, 2022

Conversation

S-YOU
Copy link
Contributor

@S-YOU S-YOU commented Nov 10, 2022

Currently it is only generate 6 words (index: 1 to 6), this fixes to use all words.

@S-YOU
Copy link
Contributor Author

S-YOU commented Nov 10, 2022

@bxcodec please take a look when you have time, currently it looks something like.

Voluptatem sit consequatur accusantium aut perferendis.
Accusantium perferendis aut consequatur sit voluptatem.
Consequatur aut voluptatem perferendis accusantium sit.
Sit aut accusantium perferendis consequatur voluptatem.
Consequatur perferendis aut accusantium voluptatem sit.
Consequatur accusantium aut voluptatem perferendis sit.
Consequatur voluptatem sit aut perferendis accusantium.
Voluptatem sit accusantium consequatur aut perferendis.
Aut sit perferendis accusantium consequatur voluptatem.
Perferendis sit voluptatem accusantium aut consequatur.
Consequatur aut accusantium sit voluptatem perferendis.
Sit accusantium voluptatem perferendis aut consequatur.
Sit accusantium perferendis consequatur aut voluptatem.
Perferendis accusantium consequatur sit aut voluptatem.
Consequatur sit voluptatem aut accusantium perferendis.
Consequatur aut sit voluptatem perferendis accusantium.
Consequatur sit accusantium voluptatem aut perferendis.
Voluptatem sit perferendis consequatur aut accusantium.
Aut voluptatem consequatur accusantium sit perferendis.
Perferendis consequatur sit aut accusantium voluptatem.
Sit perferendis accusantium consequatur aut voluptatem.
Aut consequatur perferendis sit accusantium voluptatem.
Accusantium voluptatem aut sit perferendis consequatur.
Perferendis voluptatem aut consequatur accusantium sit.
Sit accusantium aut voluptatem consequatur perferendis.
Perferendis voluptatem consequatur accusantium aut sit.
Consequatur aut perferendis voluptatem sit accusantium.
Aut accusantium consequatur sit voluptatem perferendis.
Voluptatem consequatur aut accusantium perferendis sit.
Accusantium sit perferendis aut consequatur voluptatem.
Sit accusantium aut perferendis consequatur voluptatem.
Perferendis accusantium voluptatem aut consequatur sit.
Perferendis sit consequatur accusantium voluptatem aut.
Perferendis voluptatem consequatur sit accusantium aut.
Consequatur accusantium sit aut perferendis voluptatem.
Voluptatem sit aut accusantium consequatur perferendis.
Voluptatem consequatur accusantium sit aut perferendis.
Sit accusantium voluptatem aut consequatur perferendis.
Accusantium voluptatem perferendis consequatur aut sit.

Copy link
Contributor

@bxcodec bxcodec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

LGTM

@bxcodec bxcodec merged commit 07c994c into go-faker:main Dec 7, 2022
@S-YOU S-YOU deleted the fix/sentence branch December 8, 2022 02:42
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

Successfully merging this pull request may close these issues.

2 participants