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

Propose PHP <= 5.5 #305

Merged
merged 5 commits into from
Dec 14, 2016
Merged

Propose PHP <= 5.5 #305

merged 5 commits into from
Dec 14, 2016

Conversation

infabo
Copy link
Contributor

@infabo infabo commented Dec 12, 2016

Propose PHP <= 5.5 for 1.x branch.

infabo and others added 3 commits October 24, 2016 15:57
Doctrine just launched Annotations 1.3.0 which is now PHP 5.6+.
@lisachenko
Copy link
Member

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",
Copy link
Member

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.

Copy link
Contributor Author

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?

Copy link
Member

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",
Copy link
Member

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

acknowledged.

@infabo
Copy link
Contributor Author

infabo commented Dec 13, 2016

Can't find any issues in composer.lock.

@lisachenko lisachenko merged commit d86053a into goaop:1.x Dec 14, 2016
@lisachenko
Copy link
Member

Merged, thanks

@lisachenko
Copy link
Member

Also, removed the composer.lock file from the repository.

@lisachenko lisachenko added this to the 1.2.0 milestone Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants