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

Rollback #1389 #1535

Merged
merged 2 commits into from
Feb 12, 2019
Merged

Rollback #1389 #1535

merged 2 commits into from
Feb 12, 2019

Conversation

vbrazo
Copy link
Member

@vbrazo vbrazo commented Feb 12, 2019

Rollbacks #1389

Closes #1533
Closes #1534

@vbrazo vbrazo self-assigned this Feb 12, 2019
@vbrazo vbrazo requested a review from stympy February 12, 2019 12:06
@stympy stympy merged commit 8eff383 into master Feb 12, 2019
@stympy
Copy link
Contributor

stympy commented Feb 12, 2019

@vbrazo I'll update the changelog and release 1.9.3

@vbrazo vbrazo deleted the rollback-i18n-PR branch February 12, 2019 12:19
@vbrazo
Copy link
Member Author

vbrazo commented Feb 12, 2019

sounds good. Thanks @stympy

@stympy
Copy link
Contributor

stympy commented Feb 12, 2019

Done

@henrik
Copy link

henrik commented Feb 12, 2019

Thanks so much for the quick fix ❤️

@vbrazo
Copy link
Member Author

vbrazo commented Feb 13, 2019

@henrik yeah! We needed to move fast because of the issue impact. Thanks for reporting and using Faker. We'll do more tests against those new scenarios and see if we can fix that i18n issue!

davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Rollback i18n PR

* Require helpers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants