ci: change action running schedule to 15 days interval

[alaminfirdows]

Since building and deploying the Docker image takes around 40 minutes, running the action every week quickly reaches the action usage limit. To avoid this, update the schedule to run every 15 days instead.

Summary by CodeRabbit

• New Features

  • None

• Chores

  • Streamlined deployment workflow into clear, sequential steps for building, testing, and publishing Docker images.
  • Added concurrency controls and updated scheduled triggers for more predictable runs.
  • Standardized workflow triggers and version matrices for consistency.

• Refactor

  • Reorganized CI job steps for improved readability and maintainability without changing behavior.

• Style

  • Unified quoting, indentation, and formatting across workflows; no functional impact.

[coderabbitai]

Walkthrough

Restructures deploy workflow into discrete Buildx/build/test/login/push steps, updates the schedule cron and concurrency, and normalizes quoting/indentation in both deploy and test workflows; no exported/public API changes.

Changes

Cohort / File(s)	Summary
Deploy workflow restructuring & formatting \.github/workflows/deploy.yml	Updated schedule cron to 0 0 1,15 * *, added concurrency group + cancel-in-progress, converted double-quoted strings to single-quoted, and split a monolithic build/test/push sequence into discrete steps: set up Buildx, build image, install Container Structure Test, run tests, Docker Hub login, and push. Indentation/spacing adjusted.
Test workflow concurrency & formatting \.github/workflows/test.yml	Added concurrency grouping (workflow/ref, cancel-in-progress), normalized PHP matrix quoting to single quotes, and re-indented/reflowed YAML; functional behavior remains equivalent.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Runner as GitHub Actions Runner
    participant Buildx as Docker Buildx
    participant Image as OpenLiteSpeed Image
    participant CST as Container Structure Test
    participant Hub as Docker Hub

    Note over Runner: Scheduled/Manual dispatch

    Runner->>Buildx: Set up Buildx
    Runner->>Buildx: Build image (matrix tags & args)
    Buildx-->>Runner: Built image artifact

    Runner->>CST: Install Container Structure Test
    Runner->>CST: Run structure tests against image
    CST-->>Runner: Test results

    alt tests pass
        Runner->>Hub: Login to Docker Hub
        Runner->>Hub: Push image (matrix tags)
        Hub-->>Runner: Push confirmation
    else tests fail
        Note over Runner,CST: Abort before login/push
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• fix: mysql client test #14 — Modifies GitHub Actions workflows' build/test/login/push steps and image tagging; closely related to these workflow edits.

Poem

I nibbled at YAML, neat and spry,
Split steps like carrots, stacked them high. 🥕
Buildx hummed, tests hopped in tune,
Login, push—beneath the moon.
Single quotes, tidy trail—my burrow’s new balloon.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly communicates the primary change—adjusting the CI workflow schedule to every 15 days—using a clear “ci:” prefix and no unnecessary details, which aligns with the expectations for a concise and descriptive pull request title.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 47c8918 and 28f8f03.

📒 Files selected for processing (2)

• .github/workflows/deploy.yml (3 hunks)
• .github/workflows/test.yml (3 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• .github/workflows/deploy.yml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (11)

• GitHub Check: build-php (8.2)
• GitHub Check: build-php (8.1)
• GitHub Check: build-php (8.3)
• GitHub Check: build-php (8.4)
• GitHub Check: build-php (8.0)
• GitHub Check: build-php (7.4)
• GitHub Check: build-openlitespeed (8.4, lsphp84)
• GitHub Check: build-openlitespeed (7.4, lsphp74)
• GitHub Check: build-openlitespeed (8.1, lsphp81)
• GitHub Check: build-openlitespeed (8.2, lsphp82)
• GitHub Check: build-openlitespeed (8.3, lsphp83)

🔇 Additional comments (4)

.github/workflows/test.yml (4)

9-11: LGTM! Good addition of concurrency control.

The concurrency configuration will prevent redundant test runs when new commits are pushed to the same PR, saving CI resources.

---

21-21: LGTM! Style normalization.

The change from double-quoted to single-quoted strings is a style normalization with no functional impact.

---

59-68: Style normalization looks good; verify PHP 8.0 exclusion.

The change to single-quoted strings is consistent with the style normalization. However, note that PHP 8.0 is included in the build-php matrix (line 21) but excluded from the build-openlitespeed matrix.

Please confirm whether the exclusion of PHP 8.0 from the OpenLiteSpeed builds is intentional.

---

70-98: LGTM! Indentation normalized.

The re-indentation of the OpenLiteSpeed job steps improves consistency with no functional changes.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

• Public repositories are always opted into early access features.
• You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2e5e00f and 47c8918.

📒 Files selected for processing (2)

• .github/workflows/deploy.yml (3 hunks)
• .github/workflows/test.yml (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-openlitespeed (7.4, lsphp74)

🔇 Additional comments (2)

.github/workflows/test.yml (1)

17-17: LGTM! Formatting improvements are consistent.

The changes normalize quote styles (double to single quotes) and improve indentation consistency across the workflow. No functional changes detected.

Also applies to: 55-64, 66-94

.github/workflows/deploy.yml (1)

9-9: LGTM! Formatting and workflow structure improvements.

The changes include:

• Quote normalization to single quotes for consistency
• Improved step separation: build → test → login → push sequence is now more explicit and modular
• Better indentation and alignment

These changes improve readability and maintainability without altering the functional behavior of the workflow.

Also applies to: 11-11, 21-21, 74-85, 87-137