Skip to content

Conversation

@dbu
Copy link
Contributor

@dbu dbu commented Nov 18, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets -
Documentation -
License MIT

What's in this PR?

Test with PHP 7.3
Drop tests with PHP 5

Why?

Drop PHP 5 because travis-ci does not properly support them anymore, and they are so outdated that we should not use them anymore.

Checklist

  • Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix

@dbu dbu changed the title drop php 5 because build no longer works, test with php 7.3, 7.4 drop php 5 because build no longer works, test with php 7.3 Nov 18, 2019
@dbu dbu force-pushed the php-upgrade branch 2 times, most recently from 532fe09 to 9c2fd1e Compare November 18, 2019 08:07
Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

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

We should update the branch alias in composer.json to 1.3-dev.

@dbu
Copy link
Contributor Author

dbu commented Nov 18, 2019

good point, fixed the branch alias.

@dbu dbu merged commit 6e18992 into master Nov 18, 2019
@dbu dbu deleted the php-upgrade branch November 18, 2019 08:37
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.

3 participants