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

Fix typo #453

Merged
merged 1 commit into from
Dec 17, 2019
Merged

Fix typo #453

merged 1 commit into from
Dec 17, 2019

Conversation

drupol
Copy link
Contributor

@drupol drupol commented Nov 29, 2019

This pr relates to #452 .

@drupol drupol changed the title Test using lower dependencies for php > 5.6 Test with lower dependencies for php >= 5.6 Dec 5, 2019
@drupol
Copy link
Contributor Author

drupol commented Dec 10, 2019

@ciaranmcnulty Do you have any feedback on this? Should we keep tests with php < 5.6 ? Is it still required ?

@drupol
Copy link
Contributor Author

drupol commented Dec 17, 2019

As PHPSpec 3 is in end of life, shouldn't we also update its version ?

@ciaranmcnulty
Copy link
Member

@drupol we can't update past phpspec 3 apparently because of BC breaks

@drupol drupol changed the title Test with lower dependencies for php >= 5.6 Fix typo Dec 17, 2019
@drupol
Copy link
Contributor Author

drupol commented Dec 17, 2019

@ciaranmcnulty as long as we support unsupported PHP version (< 5.6), then it's impossible to test with lowest deps on Travis.
I will revisit this as soon as we will drop those from Travis.

@ciaranmcnulty ciaranmcnulty merged commit fd4736f into phpspec:master Dec 17, 2019
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