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

Use semver for PHP version at composer.json, added support for PHP 7.3 at Travis #508

Merged
merged 1 commit into from
Mar 18, 2019

Conversation

phansys
Copy link
Contributor

@phansys phansys commented Mar 15, 2019

Q A
Branch 2.1
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

PhpredisClientFactoryTest was moved under Tests namespace, and testCreateFullConfig() test is marked with "redis4" group in order to be excluded at CI builds with Redis 4 (see #399).

@B-Galati
Copy link
Collaborator

See my comment #505 (comment)

@B-Galati B-Galati closed this Mar 15, 2019
@phansys
Copy link
Contributor Author

phansys commented Mar 15, 2019

I can keep the move of PhpredisClientFactoryTest on the other PR, but I think the semver change and the support for PHP 7.3 are a good improvement. Don't you agree?

@B-Galati
Copy link
Collaborator

@phansys You're right ;-) Thanks!

@B-Galati B-Galati reopened this Mar 15, 2019
@phansys phansys force-pushed the travis branch 3 times, most recently from d90d52c to a2fec55 Compare March 15, 2019 18:32
@phansys
Copy link
Contributor Author

phansys commented Mar 15, 2019

Great! Updated the changes to only cover these concerns.

@B-Galati B-Galati merged commit fad5047 into snc:2.1 Mar 18, 2019
@B-Galati
Copy link
Collaborator

Thanks @phansys!

@phansys phansys deleted the travis branch March 18, 2019 10:38
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