-
Notifications
You must be signed in to change notification settings - Fork 65
Update nelmio/alice to ~3.0 #249
base: master
Are you sure you want to change the base?
Conversation
|
Ok, it’s probably a good idea to require php |
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.
Thanks for this @sanpii!
Sorry, I know that this is a bit late now, so tell me if you don't have the time to address my comment atm.
composer.json
Outdated
| "behat/mink-extension": "~2.0", | ||
| "fzaninotto/faker": "~1.4", | ||
| "nelmio/alice": "~2.0", | ||
| "nelmio/alice": "~3.0", |
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.
If you put ^3.1 instead, we could make this package compatible with Symfony4.
You also need to upgrade the php minimal version to ^7.0.
|
I have nothing against this change. It break the BC. Well, this project will hardly break the BC in next versions. So I guess there's nothing really wrong with that. Also it's still a Please notice anyway that the main branch for next version is |
No description provided.