-
-
Notifications
You must be signed in to change notification settings - Fork 164
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
Propose PHP <= 5.5 #305
Propose PHP <= 5.5 #305
Conversation
Doctrine just launched Annotations 1.3.0 which is now PHP 5.6+.
Hi, there are errors in the composer.lock, please resolve them. |
@@ -8,9 +8,9 @@ | |||
"license": "MIT", | |||
|
|||
"require": { | |||
"php": ">=5.5.0", | |||
"php": "<=5.5.99", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to use ~5.5.0 here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would restrict the 1.x branch to PHP 5.5 only. Any reasons why not support PHP 5.3 / 5.4?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5.3 and 5.4 are not supported, anyone who wants to use an old version, can switch to 0.x versions.
"jakubledl/dissect": "~1.0", | ||
"doctrine/annotations": "~1.0", | ||
"doctrine/annotations": "<1.3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, switch to ~1.2.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
acknowledged.
Can't find any issues in composer.lock. |
Merged, thanks |
Also, removed the |
Propose PHP <= 5.5 for 1.x branch.