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

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
"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