Skip to content
This repository has been archived by the owner on Sep 15, 2022. It is now read-only.

Update nelmio/alice to ~3.0 #249

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update nelmio/alice to ~3.0 #249

wants to merge 5 commits into from

Conversation

sanpii
Copy link

@sanpii sanpii commented Feb 15, 2018

No description provided.

@sanpii
Copy link
Author

sanpii commented Feb 15, 2018

Ok, it’s probably a good idea to require php >=7.0 before merging my PR.

Copy link
Collaborator

@pimolo pimolo left a 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",
Copy link
Collaborator

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.

@Nek-
Copy link
Contributor

Nek- commented Jul 25, 2018

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 0.x version that allow us to do that.

Please notice anyway that the main branch for next version is feature/1.0 (but is probably not functional at the moment)

@pimolo pimolo mentioned this pull request Sep 9, 2018
tkulka added a commit to blossom-educational/FriendlyContexts that referenced this pull request Dec 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants