Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Php optimize#3165

Merged
NateBrady23 merged 3 commits intoTechEmpower:masterfrom
joanhey:php-optimize
Jan 9, 2018
Merged

Php optimize#3165
NateBrady23 merged 3 commits intoTechEmpower:masterfrom
joanhey:php-optimize

Conversation

@joanhey
Copy link
Copy Markdown
Contributor

@joanhey joanhey commented Jan 5, 2018

Updated to PHP 7.2.1
The MCrypt extension has now been moved out of the core.
Udpated Phalcon and Yaf to work with php 7.2

Deleted mcript module in php 7.2.1
Updated Readmes
@mention-bot
Copy link
Copy Markdown

Thanks @joanhey for contributing to The Framework Benchmarks! @TechEmpower, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

@joanhey
Copy link
Copy Markdown
Contributor Author

joanhey commented Jan 5, 2018

Install Mcrypt via pecl. (Not recommended)
Until the old frameworks update.

Framework Version bench Version without Mcrypt Actual Version
CakePHP 2.10.4 3.0 3.5
Laravel 4.2 5.1 5.5

Or we can leave only for PHP 5 these two old frameworks, until they update.
So they will have an incentive to upgrade.
What I do ??

@joanhey
Copy link
Copy Markdown
Contributor Author

joanhey commented Jan 8, 2018

As no one has answered.
I will use the second option

Until they upgrade the version
@NateBrady23
Copy link
Copy Markdown
Member

Ah perfect @joanhey that's what I would have recommended

@NateBrady23 NateBrady23 merged commit 4cc5c07 into TechEmpower:master Jan 9, 2018
@joanhey joanhey deleted the php-optimize branch January 9, 2018 17:18
orangy pushed a commit to orangy/FrameworkBenchmarks that referenced this pull request Jan 22, 2018
* Git ignore .vscode

* Updated: PHP 7.2.1, Phalcon 3.24, yaf 3.0.6
Deleted mcript module in php 7.2.1
Updated Readmes

* Laravel and Cakephp only php5
Until they upgrade the version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants