-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Remove nette/utils to allow easier install #86
Conversation
TomasVotruba
commented
Jul 31, 2023
- make use of preg_match_all()
- use Arrays::flatten() locally
98425fe
to
3ddf340
Compare
4df9661
to
56ad38d
Compare
"phpstan/phpstan": "^1.10.19", | ||
"nette/utils": "^3.2", |
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.
great move to drop the dependency
Hi, in |
@LeoVie Hi, yes, great catch! Could you send PR to remove it please? |
Sure. |
Is there a reason for |
Not really, should be removed too |