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

Correct Faker::Address.city and Faker::Address.street_address data for :fr locale #3018

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

brupla6126
Copy link

Motivation / Background

Our Rails app test suite validations are frequently failing when using :fr locale due to
1 - Numbers in city names
2 - Extra space in street names

Faker::Address.city

  • Dumbéa11
  • Dunkerque14

Faker::Address.street_address

  • Leo Bleriot with 2 spaces

@stefannibrasil
Copy link
Contributor

hi @brupla6126 do you mind running this formatter first? https://github.com/faker-ruby/faker/blob/main/CONTRIBUTING.md#yaml-files without this format, it's really hard to review this. This is one of the files that are missing this reformat. Thank you!

@brupla6126 brupla6126 force-pushed the fix_fr_address_city_and_street_translations branch from b651d6c to 55fad8c Compare November 7, 2024 12:02
@brupla6126
Copy link
Author

brupla6126 commented Nov 7, 2024

hi @brupla6126 do you mind running this formatter first? https://github.com/faker-ruby/faker/blob/main/CONTRIBUTING.md#yaml-files without this format, it's really hard to review this. This is one of the files that are missing this reformat. Thank you!

Hey @stefannibrasil I formatted the file but I noticed that many other locale files are unformatted. Maybe a rake task that will format all locale files at once?

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