Updated CakePHP version #2

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@josegonzalez

Your benchmark would be more useful with recent versions of the framework. Just helping out.

@seedifferently
Owner

Thanks Jose!

I thought I might just leapfrog 2.0 and jump to 2.1 when it's released. I am planning an update across the board soon but am just waiting for a few releases (Django 1.4, Pyramid 1.3, CodeIgniter 2.1, etc).

@josegonzalez

Same exact code will work in 2.1. No changes necessary. Just update your benchmarks when it comes out.

@seedifferently
Owner

Great, thanks again!

@josegonzalez

Hint, you can merge this pull request in.

@ryansnowden

I'm quite happy with that Hint

@josegonzalez

Will update the pull request tonight to fix the core.php to turn off all level of debugging, logging, etc.

@seedifferently
Owner

If we're thinking "production mode" with these tests, wouldn't a certain level of logging still be enabled?

@ryansnowden

Only during beta. PHP, Apache and MySQL provide the necessary. Logging gets turned on to get more info if needed.

@seedifferently
Owner

Fair enough.

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