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

[php] Update Codeigniter to v 4#7686

Merged
NateBrady23 merged 4 commits intoTechEmpower:masterfrom
joanhey:codeigniter4
Nov 16, 2022
Merged

[php] Update Codeigniter to v 4#7686
NateBrady23 merged 4 commits intoTechEmpower:masterfrom
joanhey:codeigniter4

Conversation

@joanhey
Copy link
Copy Markdown
Contributor

@joanhey joanhey commented Nov 10, 2022

It's a complete rewrite because it's BC, and added update test.

#6894

@joanhey joanhey mentioned this pull request Nov 10, 2022
17 tasks
@joanhey
Copy link
Copy Markdown
Contributor Author

joanhey commented Nov 11, 2022

Performance info

CI3 vs CI4

Framework JSON 1-query 20-query Fortunes Updates Plaintext
CI3 26,313 25,189 12,049 24,358 -- 25,646
CI4 5,746 5,314 4,244 5,320 2,086 5,791

@NateBrady23 NateBrady23 merged commit 297d1e4 into TechEmpower:master Nov 16, 2022
@joanhey joanhey deleted the codeigniter4 branch November 17, 2022 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants