Skip to content

remove php 8.5 support for OLS due to build failure#31

Merged
itsrafsanjani merged 1 commit intoflywp:mainfrom
ma-04:fix/remove_ols_8.5_support
Feb 16, 2026
Merged

remove php 8.5 support for OLS due to build failure#31
itsrafsanjani merged 1 commit intoflywp:mainfrom
ma-04:fix/remove_ols_8.5_support

Conversation

@ma-04
Copy link
Member

@ma-04 ma-04 commented Feb 16, 2026

OLS missing packages causing build failure for php 8.5.

#9 9.302 Building dependency tree...
#9 9.521 Reading state information...
#9 9.537 E: Unable to locate package lsphp85-opcache
#9 ERROR: process "/bin/sh -c wget -O - https://repo.litespeed.sh/ | bash     && apt-get update && apt-get install -y --no-install-recommends     ${LSPHP} ${LSPHP}-common ${LSPHP}-mysql     ${LSPHP}-opcache ${LSPHP}-curl ${LSPHP}-intl     ${LSPHP}-imagick ${LSPHP}-redis ${LSPHP}-igbinary     ${LSPHP}-imap ${LSPHP}-ldap &&     ln -s /usr/local/lsws/$LSPHP/bin/php /usr/bin/php &&     ln -s /usr/local/lsws/$LSPHP/etc/php/$PHP_VERSION/mods-available/ /usr/local/lsws/php-mods-available &&     rm -rf /var/lib/apt/lists/* &&     apt-get clean && apt-get autoremove -y" did not complete successfully: exit code: 100

Summary by CodeRabbit

  • Chores
    • Removed PHP 8.5 from build and test pipeline configurations.

@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

Removes PHP version 8.5 from GitHub Actions deploy workflow matrices and their corresponding lsphp mapping entries, eliminating 8.5 from all build and test paths in the CI/CD configuration.

Changes

Cohort / File(s) Summary
PHP Version Configuration
.github/workflows/deploy.yml
Removed PHP 8.5 from both build-php and build-openlitespeed matrix configurations, along with associated lsphp mapping entries for version 8.5.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • alaminfirdows

Poem

🐰 Eight-point-five hops away so fast,
From CI/CD pipelines of the past,
We trim the matrix, lean and true,
Workflows dance in lighter hue,
Simplicity wins the sprint today! 🚀

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: removing PHP 8.5 support for OpenLiteSpeed due to build failure, which matches the changeset's core objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@itsrafsanjani itsrafsanjani merged commit 4c735df into flywp:main Feb 16, 2026
6 of 13 checks passed
@ma-04 ma-04 deleted the fix/remove_ols_8.5_support branch February 16, 2026 10:13
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.

2 participants