Skip to content

Deprecate zh_TW methods that only call base methods#122

Merged
pimjansen merged 3 commits intoFakerPHP:mainfrom
krsriq:deprecate_zh_tw_methods
Dec 20, 2020
Merged

Deprecate zh_TW methods that only call base methods#122
pimjansen merged 3 commits intoFakerPHP:mainfrom
krsriq:deprecate_zh_tw_methods

Conversation

@krsriq
Copy link
Copy Markdown

@krsriq krsriq commented Dec 8, 2020

What is the reason for this PR?

In zh_TW there are two classes that use Factory::create('en_US') only to call base methods.

  • A new feature
  • Fixed an issue (resolve #ID)

Author's checklist

Summary of changes

This PR deprecates these classes and methods. For BC the respective parent methods are called.

Review checklist

  • All checks have passed
  • Changes are approved by maintainer

@krsriq krsriq changed the title deprecate zh_TW methods that only call base methods Deprecate zh_TW methods that only call base methods Dec 8, 2020
@GrahamCampbell GrahamCampbell requested a review from Nyholm December 8, 2020 17:18
@bram-pkg
Copy link
Copy Markdown
Member

bram-pkg commented Dec 9, 2020

Looks good to me.

Comment thread src/Faker/Provider/zh_TW/Payment.php Outdated
@pimjansen pimjansen merged commit a9498d8 into FakerPHP:main Dec 20, 2020
@krsriq
Copy link
Copy Markdown
Author

krsriq commented Dec 20, 2020

Thanks!

@krsriq krsriq deleted the deprecate_zh_tw_methods branch December 20, 2020 12:48
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.

4 participants