-
Notifications
You must be signed in to change notification settings - Fork 504
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
[1.10.x] Create DirectoryCreator helper class #2697
Conversation
You've opened the pull request against the latest branch 1.11.x. If your code is relevant on 1.10.x and you want it to be released sooner, please rebase your pull request and change its target to 1.10.x. |
As far as PHP is concerned, this code is equivalent. You don't need to check is_dir before calling mkdir, the current code is fine. I suspect the problem has a different root cause in Pest which should be addressed instead. |
Sure, I get that---likely having to do with how it's capturing warnings. But there's still three other calls to |
If we wanted to make this consistent everywhere, I'd welcome a new class |
@ondrejmirtes done---please let me know if you'd like any other changes |
5ac3465
to
0d8a040
Compare
d223897
to
1bbe0f2
Compare
12dbf26
to
91d88e2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After this it's good to merge 👍
Thank you. |
Just did this to improve compatibility with PHPUnit and PHP-Parser 5: 9dac90d |
Hello!
This is similar to #572 --- every other
mkdir
call is guarded by anis_dir
check so I updated this to be consistent.Thanks!