search

cakephp / app

CakePHP application template
370 stars 391 forks source link

Update Application.php to remove route cache code comment #909

Closed umer936 closed 2 years ago

umer936 commented 2 years ago

Also adds note on plugin to add cached routing

Unsure if this PR should be against 4.x branch or 4.next. I can resubmit if it's the latter.

othercorey commented 2 years ago

The target branch should match the cakephp version it targets. If the target was already released then it should go into 4.x.

umer936 commented 2 years ago

Then I believe it is correct to go into 4.x as the code change removing the 2nd parameter was previously committed.