Skip to content

Added missing method for Hungarian married female lastnames#503

Merged
pimjansen merged 1 commit intoFakerPHP:mainfrom
pimjansen:feature/hungarian-female-name
Jul 22, 2022
Merged

Added missing method for Hungarian married female lastnames#503
pimjansen merged 1 commit intoFakerPHP:mainfrom
pimjansen:feature/hungarian-female-name

Conversation

@pimjansen
Copy link
Copy Markdown

@pimjansen pimjansen commented Jul 21, 2022

What is the reason for this PR?

Author's checklist

Summary of changes

Added missing method

Review checklist

  • All checks have passed
  • Changes are approved by maintainer

@pimjansen pimjansen added the bug Something isn't working label Jul 21, 2022
@pimjansen pimjansen changed the title Added missing method for Hungarian female firstnames Added missing method for Hungarian married female lastnames Jul 21, 2022
Copy link
Copy Markdown

@mmarton mmarton left a comment

Choose a reason for hiding this comment

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

LGTM

@bram-pkg
Copy link
Copy Markdown
Member

@pimjansen quick tests maybe?

@pimjansen
Copy link
Copy Markdown
Author

@pimjansen quick tests maybe?

Yeah didnt do it yet since there are none for this Locale at all and had some time constraints.

Will add them

@pimjansen
Copy link
Copy Markdown
Author

@bram-pkg added basic test

@bram-pkg
Copy link
Copy Markdown
Member

Just the cs-fixer check is failing now

@pimjansen
Copy link
Copy Markdown
Author

Just the cs-fixer check is failing now

Yeah booboo as usual 😭

@pimjansen pimjansen merged commit 2620f58 into FakerPHP:main Jul 22, 2022
@pimjansen pimjansen deleted the feature/hungarian-female-name branch July 22, 2022 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hungarian name generation is broken

3 participants