Skip to content

ci: bump min php version to 8.3#768

Open
lotyp wants to merge 1 commit into
masterfrom
ci/bump-to-php83
Open

ci: bump min php version to 8.3#768
lotyp wants to merge 1 commit into
masterfrom
ci/bump-to-php83

Conversation

@lotyp

@lotyp lotyp commented Jun 1, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated PHP version from 8.2 to 8.3 across all CI/CD workflows (coding standards, dependency analysis, security analysis, static analysis, and testing pipelines).

@vercel

vercel Bot commented Jun 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
laravel-cycle-orm-adapter Ready Ready Preview, Comment Jun 1, 2026 6:14pm

@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@lotyp, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 56 minutes and 35 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 63611e85-2ef7-4939-846d-067cb11c991b

📥 Commits

Reviewing files that changed from the base of the PR and between 9e24a74 and 624a7fc.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (9)
  • .github/workflows/coding-standards.yml
  • .github/workflows/dependency-analysis.yml
  • .github/workflows/security-analysis.yml
  • .github/workflows/static-analysis.yml
  • .github/workflows/testing-mssql.yml
  • .github/workflows/testing-mysql.yml
  • .github/workflows/testing-pgsql.yml
  • .github/workflows/testing.yml
  • composer.json
📝 Walkthrough

Walkthrough

All CI workflow files update their PHP version matrices from 8.2 to 8.3. Changes span code quality checks, dependency and security analysis, and multi-database integration tests. Most workflows replace the single matrix entry; the SQLite test suite removes 8.2 while keeping 8.3 and 8.4.

Changes

CI PHP 8.3 Upgrade

Layer / File(s) Summary
Code Quality and Standards Workflows
.github/workflows/coding-standards.yml, .github/workflows/static-analysis.yml
Composer linting and coding standards jobs update PHP matrix from 8.2 to 8.3, maintaining all quality check steps and tooling.
Dependency and Security Analysis Workflows
.github/workflows/dependency-analysis.yml, .github/workflows/security-analysis.yml
Dependency analysis and security audit jobs update PHP matrix from 8.2 to 8.3 while preserving validation and audit steps.
Database Integration Test Workflows
.github/workflows/testing-mssql.yml, .github/workflows/testing-mysql.yml, .github/workflows/testing-pgsql.yml
MSSQL, MySQL, and PostgreSQL test jobs update PHP matrix from 8.2 to 8.3, keeping database service and integration test execution unchanged.
Core Test Suite Updates
.github/workflows/testing.yml
Code coverage, mutation testing, and SQLite test jobs update PHP matrix to 8.3; SQLite matrix removes 8.2 entirely while retaining 8.3 and 8.4.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

type: maintenance

Poem

🐰 A version hops along the way,
From eight-point-two to eight-point-three today!
Each workflow's matrix shifts with grace,
New PHP standards set the pace. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: bump min php version to 8.3' directly and clearly describes the main change—updating CI workflows to require PHP 8.3 as the minimum version.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/bump-to-php83

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.

@github-actions github-actions Bot added the type: maintenance For maintenance, refactor and testing (perf, chore, style, revert, refactor, test, build, ci) label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: maintenance For maintenance, refactor and testing (perf, chore, style, revert, refactor, test, build, ci)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant