Skip to content

add and fix php 7.3#593

Merged
nicolasmure merged 3 commits intomasterfrom
feature/php7.3-support
Dec 13, 2018
Merged

add and fix php 7.3#593
nicolasmure merged 3 commits intomasterfrom
feature/php7.3-support

Conversation

@nicolasmure
Copy link
Copy Markdown
Contributor

I cherry-picked the commit from #587 cc/ @andreybolonin
Fixes #592 .

.travis.yml Outdated
@@ -30,6 +30,7 @@ matrix:
- php: '7.0'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should also drop support for PHP 5.6 and 7.0, as 7.0 has been EOL'ed and 5.6 will be at the end of the year.

@nicolasmure nicolasmure force-pushed the feature/php7.3-support branch 2 times, most recently from 5f96dfc to 43835f7 Compare December 10, 2018 10:44
Also bump phpunit to v7 and phpspec to v5
@nicolasmure nicolasmure force-pushed the feature/php7.3-support branch from 43835f7 to 154cc1c Compare December 10, 2018 10:56
Copy link
Copy Markdown
Contributor

@akerouanton akerouanton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@nicolasmure nicolasmure force-pushed the feature/php7.3-support branch from 154cc1c to b54a01e Compare December 13, 2018 08:57
@nicolasmure nicolasmure merged commit d0ede04 into master Dec 13, 2018
@nicolasmure nicolasmure deleted the feature/php7.3-support branch December 13, 2018 09:30
@nicolasmure nicolasmure mentioned this pull request Dec 13, 2018
@deguif
Copy link
Copy Markdown
Contributor

deguif commented Dec 19, 2018

Is a new release planned with these fixes?

@nicolasmure
Copy link
Copy Markdown
Contributor Author

Yes it is (see #597), we'll try to also make a new release of the bundle.

@deguif
Copy link
Copy Markdown
Contributor

deguif commented Dec 19, 2018

Nice, thank you very much @nicolasmure

@nicolasmure
Copy link
Copy Markdown
Contributor Author

@deguif v0.8.0 is now out ;)

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.

4 participants