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

Update required PHP version #87

Merged
merged 1 commit into from
May 24, 2016
Merged

Update required PHP version #87

merged 1 commit into from
May 24, 2016

Conversation

Nyholm
Copy link
Contributor

@Nyholm Nyholm commented May 24, 2016

Since cfd03d3 we do no longer support php 5.5

Since lcobucci@cfd03d3 we do no longer support php 5.5
@Ocramius Ocramius self-assigned this May 24, 2016
@Ocramius Ocramius added this to the 4.0.0 milestone May 24, 2016
@Ocramius Ocramius merged commit 5cfa08c into lcobucci:master May 24, 2016
@Ocramius
Copy link
Collaborator

👍

@Nyholm
Copy link
Contributor Author

Nyholm commented May 24, 2016

Thank you for merging

@Nyholm Nyholm deleted the patch-1 branch May 24, 2016 16:11
@lcobucci
Copy link
Owner

Actually this is not fully correct 😄 on v3.x we still support PHP 5.5+.
The PHP 7 requirement is just for the next major release (master branch while is unstable).

The statement on /~https://github.com/lcobucci/jwt/blob/master/README.md#dependencies is correct so maybe we don't need that duplicated information.

@lcobucci
Copy link
Owner

But thanks @Nyholm for your contribution

@lcobucci
Copy link
Owner

I'll update that on #74 😉

@Ocramius
Copy link
Collaborator

Argh, sorry for the over-eager merge then :(

@lcobucci
Copy link
Owner

@Ocramius relax, no harm done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants