-
Notifications
You must be signed in to change notification settings - Fork 363
Update minimum PHP version to 5.3.2 #96
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
Update minimum PHP version to 5.3.2 #96
Conversation
|
We may as well bump it to 5.3.3 |
|
Bumped to 5.3.3 (note that this is the version used by RHEL 6 so hopefully min PHP version won't go beyond 5.3.3) |
|
👎 Composer depends on this package. |
|
I'm fine with 5.3.2 but was requested to change it to 5.3.3. Whichever the maintainers prefer I will make it so. |
1e5f43c to
3e45ca4
Compare
|
Rebased and repushed. |
3e45ca4 to
cffdeb7
Compare
bin/validate-json uses stream_resolve_include_path() which is only available in PHP >= 5.3.2
cffdeb7 to
d81a42b
Compare
|
Re-based to current master and reverted commits to original pull request of min PHP version of 5.3.2 |
|
👍 |
|
Also, composer itself depends on php 5.3.2+, so you literally can't install this package on php 5.3.0... |
|
Could we get this merged? |
|
+1 |
Update minimum PHP version to 5.3.2
bin/validate-jsonusesstream_resolve_include_path()which is only available in PHP >= 5.3.2