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

Bump min PHP to 7.0, start running tests on 7.1 and 7.2 #217

Merged
merged 1 commit into from Mar 21, 2018

Conversation

bookernath
Copy link
Contributor

@bookernath bookernath commented Mar 21, 2018

PHP 7.0 is the lowest supported version these days: http://php.net/supported-versions.php

Therefore, I've updated readmes and other things to remove support for anything lower.

I've also added 7.1 and 7.2 to Travis so we can start testing for compatibility with modern versions.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 69.153% when pulling beef683 on bookernath:bump-min-php-56 into e57fc7a on bigcommerce:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 69.153% when pulling beef683 on bookernath:bump-min-php-56 into e57fc7a on bigcommerce:master.

@coveralls
Copy link

coveralls commented Mar 21, 2018

Coverage Status

Coverage increased (+1.4%) to 70.557% when pulling 6800595 on bookernath:bump-min-php-56 into 6016f47 on bigcommerce:master.

@bookernath
Copy link
Contributor Author

Plan is to follow up with actual fixes for 7.1 and 7.2.

@lord2800
Copy link
Contributor

Given http://php.net/supported-versions.php, I'd say we should just drop support for 5.6 now and call it good.

@bookernath bookernath force-pushed the bump-min-php-56 branch 2 times, most recently from b7f95e7 to fba56e9 Compare March 21, 2018 17:34
@bookernath bookernath changed the title Bump min PHP to 5.6, start running tests on 7.1 and 7.2 Bump min PHP to 7.0, start running tests on 7.1 and 7.2 Mar 21, 2018
@bookernath
Copy link
Contributor Author

Updated to 7.0 @lord2800

@bookernath
Copy link
Contributor Author

7.0 passed, 7.1 and 7.2 have one failing test, follow-up PR will fix those.

PHP 5.6 is the lowest supported version these days: http://php.net/supported-versions.php

Therefore, I've updated readmes and other things to remove support for anything lower.

I've also added 7.1 and 7.2 to Travis so we can start testing for compatibility with modern versions.
@bookernath bookernath merged commit ce672e6 into bigcommerce:master Mar 21, 2018
@bookernath bookernath deleted the bump-min-php-56 branch March 21, 2018 17:46
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.

None yet

3 participants