Skip to content
This repository has been archived by the owner on Feb 6, 2020. It is now read-only.

added php72 travis #201

Closed
wants to merge 2 commits into from

Conversation

kokspflanze
Copy link
Contributor

added php72 travis in build matrix

.travis.yml Outdated
@@ -38,6 +38,15 @@ matrix:
- php: 7.1
env:
- DEPS=latest
- php: 7.2
Copy link
Member

Choose a reason for hiding this comment

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

IMHO, PHP 7.2 should be added instead of "nightly"

Copy link
Member

Choose a reason for hiding this comment

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

It has 7.2 AND nightly. Nightly refers to 7.3 at the moment. Not sure if 7.3 is already a travis php tag.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, if it refers to PHP 7.3 it's good to have then !

Copy link
Member

Choose a reason for hiding this comment

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

weierophinney added a commit that referenced this pull request Nov 27, 2017
weierophinney added a commit that referenced this pull request Nov 27, 2017
Forward port #201
Forward port #202

Conflicts:
	.travis.yml
@kokspflanze kokspflanze deleted the php72-travis branch November 27, 2017 18:13
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