Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Various locale specific and other fixes for faker #2339

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

psibi
Copy link
Member

@psibi psibi commented Jun 21, 2021

Summary of the changes:

  • touhou.yml: The current file isn't a valid yaml spec. Use single quote to fix it.
  • es-AR: Remove redundant entires, the country_code should be an array (as consistent with the other formats)
  • fr: Break appliance.yml from pokemon.yml as it's a separate one. (Also makes it consistent with the en locale)
  • ja: Break animal.yml, cat.yml, dog.yml, overwatch.yml, pokemon.yml, super_mario.yml, zelda.yml from creature.yml and games.yml. This makes it consistent with the en locale too.
  • en/music.yml: Move hiphop under music as that's the top level attribute. This also makes it consistent with the other yml convention we follow.

Sidenote: These fixes were a result of work here: fakedata-haskell/fakedata#37

Summary of the changes:

- touhou.yml: The current file isn't a valid yaml spec. Use single
  quote to fix it.
- es-AR: Remove redundant entires, the `country_code` should be an
array (as consistent with the other formats)
- fr: Break `appliance.yml` from `pokemon.yml` as it's a separate
one. (Also makes it consistent with the en locale)
- ja: Break `animal.yml`, `cat.yml`, `dog.yml`, `overwatch.yml`,
`pokemon.yml`, `super_mario.yml`, `zelda.yml` from `creature.yml` and
`games.yml`. This makes it consistent with the `en` locale too.
- en/music.yml: Move hiphop under music as that's the top level
attribute. This also makes it consistent with the other yml convention
we follow.
@psibi psibi force-pushed the upstream-locale-fixes branch from e3b1e70 to ab88383 Compare June 21, 2021 10:40
@koic koic merged commit 53ead39 into faker-ruby:master Jun 22, 2021
@koic
Copy link
Member

koic commented Jun 22, 2021

Nice! 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants