-
Notifications
You must be signed in to change notification settings - Fork 66
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
@@ -12,7 +12,7 @@ | |||
"doctrine/data-fixtures": "~1.0", | |||
"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.