Skip to content
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

Update phpspec to ^6.0 version #210

Closed
wants to merge 1 commit into from

Conversation

Nek-
Copy link
Contributor

@Nek- Nek- commented May 26, 2020

Problem

phpspec was set in version ^4. This version was not compatible with
PHP 7.4. So if you have the latest version of PHP on your computer you
cannot develop on this plugin.

And the upgrade was blocking because of another dependency.

Solution

Update phpspec to ^6 version. Change phpspec-code-coverage to the
friends-of-phpspec version which allows us to use phpspec ^6.

Bonus

Add php 7.4 on travis ci.

@Nek- Nek- requested a review from a team as a code owner May 26, 2020 09:50
@Nek- Nek- changed the title fix(test): update phpspec to 6.0 version Update phpspec to 6.0 version May 26, 2020
@Nek- Nek- changed the title Update phpspec to 6.0 version Update phpspec to ^6.0 version May 26, 2020
@Nek- Nek- force-pushed the feature/fix-php74-forbidden branch from 77ca9b7 to e6fdc50 Compare May 26, 2020 11:00
@Nek- Nek- mentioned this pull request May 26, 2020
Problem
-------

phpspec was set in version ^4. This version was not compatible with
PHP 7.4. So if you have the latest version of PHP on your computer
you
cannot develop on this plugin.

And the upgrade was blocking because of another dependency.

Solution
--------

Update phpspec to ^6 version. Change phpspec-code-coverage to the
friends-of-phpspec version which allows us to use phpspec ^6.
@Nek- Nek- force-pushed the feature/fix-php74-forbidden branch from e6fdc50 to a689354 Compare May 26, 2020 16:14
@GSadee
Copy link
Member

GSadee commented Jan 7, 2021

Closing as it has been done in #205

@GSadee GSadee closed this Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants