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 php-jwt to 5.0 #61

Merged
merged 1 commit into from Jul 7, 2017
Merged

Update php-jwt to 5.0 #61

merged 1 commit into from Jul 7, 2017

Conversation

BafS
Copy link
Contributor

@BafS BafS commented Jul 5, 2017

Hi,

php-jwt does not have BC and all the tests pass, I just updated the composer for the version 5.

@BafS
Copy link
Contributor Author

BafS commented Jul 5, 2017

FYI the tests pass but the HHVM test does not pass because "HHVM is no longer supported on Ubuntu Precise. Please consider using Trusty with dist: trusty." on Travis.

@tuupola
Copy link
Owner

tuupola commented Jul 7, 2017

You mean it does not have a BC break? I was looking for a reason why they pumped up the main version number. It usually means there is a BC break, but maybe they are just not following semantic versioning.

@tuupola tuupola merged commit 7867165 into tuupola:2.x Jul 7, 2017
@tuupola
Copy link
Owner

tuupola commented Jul 7, 2017

Merged, thanks!

@BafS
Copy link
Contributor Author

BafS commented Jul 7, 2017

@tuupola Yes I think that they don't follow the semantic versioning.
Thanks for your work :) !

@tuupola
Copy link
Owner

tuupola commented Jul 12, 2017

Released as 2.3.3.

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

2 participants