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

PHP7 support #635

Merged
merged 1 commit into from
Sep 5, 2015
Merged

PHP7 support #635

merged 1 commit into from
Sep 5, 2015

Conversation

enov
Copy link
Contributor

@enov enov commented Sep 5, 2015

No description provided.

acoulton added a commit that referenced this pull request Sep 5, 2015
@acoulton acoulton merged commit c53d290 into 3.3/develop Sep 5, 2015
@enov enov deleted the 3.3/feat/php7-support branch September 18, 2015 06:10
@neo22s
Copy link
Member

neo22s commented Oct 16, 2015

hello! any news on this? how can I help? what is uncompatible?

so far only found an issue on the pagination module, working on it.

@neo22s
Copy link
Member

neo22s commented Oct 19, 2015

Seems also theres an issue with the kohana/kohana/exception.php

response(Exception $e) doesnt work but response($e) it does.

@enov
Copy link
Contributor Author

enov commented Oct 19, 2015

@neo22s this PR is already merged and only adds PHP 7.0 to Travis CI build matrix.

If Kohana has a problem running on PHP 7.0, kindly open a new issue. Also, kindly provide enough details to reproduce the issue.

Thanks :)

@neo22s
Copy link
Member

neo22s commented Oct 19, 2015

Sure! Should I open it in the core? no problem, as far as I have tested works great all except this 2 minor issues ;)

@neo22s
Copy link
Member

neo22s commented Oct 19, 2015

#642

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.

3 participants