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.

Sign up for GitHub

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

Jump to bottom

Fix: Avoid ternary statement #600

Merged

bram-pkg merged 1 commit into FakerPHP:main from localheinz:fix/ternary

Feb 2, 2023

Member

localheinz commented Feb 2, 2023 •

edited

Loading

What is the reason for this PR?

avoids nested ternary statements

Author's checklist

Follow the Contribution Guide
New features and changes are documented

Summary of changes

Review checklist

All checks have passed
Changes are approved by maintainer


          Fix: Avoid ternary statement

17457ef

bram-pkg approved these changes

View reviewed changes

bram-pkg merged commit abc97b5 into FakerPHP:main

localheinz deleted the fix/ternary branch

February 2, 2023 09:20

Member Author

localheinz commented Feb 2, 2023

Thank you, @bram-pkg!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet