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

Add Codeigniter 3.1 #15

Closed
wants to merge 1 commit into from
Closed

Add Codeigniter 3.1 #15

wants to merge 1 commit into from

Conversation

joanhey
Copy link
Contributor

@joanhey joanhey commented Jul 22, 2023

CI4 is five times slower than CI3, and they need to fix this performance regression.

Also I know this bench, because one notification in the issue that I created in CI4 about that regression.

CI3 don't work with php8.2 (exist a PR that fix that, but it isn't merged yet), but anybody can try it with with other versions.
Will be very useful for the CI core developers.

The problem, is that we need to change the parse_results() to by pass any framework that fail in any version.

@joanhey
Copy link
Contributor Author

joanhey commented Jul 22, 2023

Later I'll try to add GHactions tests.

@myaaghubi
Copy link
Owner

Not a PR but PRs, if the speed between CI3/CI4 is different then that's useful to have CI3 in the table too, however the latest results are for PHP 8.2, so official support for it is necessary. Consider the right version after PHP 8.2 get supported

@joanhey
Copy link
Contributor Author

joanhey commented Jul 23, 2023

We can check first the PHP version, and if PHP8.2 by pass CI3 for now, till they merge the fix.

@joanhey
Copy link
Contributor Author

joanhey commented Jul 24, 2023

When this PR bcit-ci/CodeIgniter#6173 is merged.
We can add CI3, and check the performance regression.

@myaaghubi myaaghubi changed the base branch from main to dev August 1, 2023 07:19
@myaaghubi myaaghubi deleted the branch myaaghubi:v2.3 August 2, 2023 15:41
@myaaghubi myaaghubi closed this Aug 2, 2023
@myaaghubi myaaghubi reopened this Aug 2, 2023
@myaaghubi
Copy link
Owner

We moved to PHP-8.3 and CI-3.1 not support it

@myaaghubi myaaghubi closed this Dec 5, 2023
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.

None yet

2 participants