[CI] Replace the full build for 2.0 with the build for 2.2 branch + reduce number of builds by GSadee · Pull Request #18431 · Sylius/Sylius

GSadee · 2025-10-10T05:00:25Z

Q	A
Branch?	2.1
Bug fix?	no
New feature?	no
BC breaks?	no
Deprecations?	no
Related tickets
License	MIT

Summary by CodeRabbit

Chores
- Updated CI workflow naming to 2.2 and adjusted daily run schedules (shifted daily run times across workflows).
- Aligned CI branch matrix to target the 2.2 line across jobs.
Tests
- Standardized test matrix to favor PHP 8.3 (with 8.4 coverage), removing PHP 8.2 in most suites.
- Revised end-to-end combinations for MySQL, MariaDB, and PostgreSQL to match new PHP/Symfony baselines and added targeted includes to preserve key coverage.

coderabbitai · 2025-10-10T05:00:52Z

Walkthrough

Updates to GitHub Actions CI configuration: adjusted scheduled cron times, renamed one workflow to "2.2", changed workflow branch matrix entries to "2.2", and standardized PHP version entries in matrix.json (promoting/removing 8.2, consolidating on 8.3 and 8.4). No job logic or steps were added or removed.

Changes

Cohort / File(s)	Change summary
CI schedule adjustment `.github/workflows/ci__full_2_1.yaml`	Cron changed from `0 3 * * ` to `0 2 * *`.
Workflow rename & branch matrix `.github/workflows/ci__full_2_2.yaml`	Workflow name updated to "Continuous Integration 2.2 (Full)"; cron changed from `0 2 * * ` to `0 3 * *`; matrix branch entries updated from `["2.0"]` to `["2.2"]` for static checks, e2e (MariaDB/MySQL/PostgreSQL), and packages.
Matrix JVM/PHP standardization `.github/workflows/matrix.json`	Removed PHP `8.2` entries, promoted `8.3` (with Symfony 6.4 baseline in some entries) and retained `8.4`; consolidated full static checks to `["8.3","8.4"]`; adjusted e2e/package matrices and added specific include variants for full/e2e-mysql.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

[CI] Update branches for full builds + refactor and upmerge actions #17887 — Modifies CI workflow files including ci__full_2_1.yaml; overlaps on schedule and workflow adjustments.
[CI] Update matrix for 2.1 branch + fix test installer #18088 — Updates .github/workflows/matrix.json and other CI configuration; related matrix changes.

Suggested reviewers

GSadee
mpysiak
TheMilek

Poem

I twitch my whiskers at the nightly run,
Cron hops back an hour to greet the sun. 🌙
PHP 8.3 bounds in, 8.4 stays near,
Workflows renamed, the matrix clear.
A carrot cheer for CI — quick and fun! 🥕

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 clearly communicates the main CI change by specifying the replacement of the 2.0 full build with the 2.2 branch and noting the reduction in the number of builds, which directly reflects the modifications made in this PR.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

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 7edcd0b and c5e740f.

📒 Files selected for processing (2)

.github/workflows/ci__full_2_1.yaml (1 hunks)
.github/workflows/matrix.json (7 hunks)

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

.github/workflows/ci__full_2_1.yaml

⏰ 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: End-to-end tests (MySQL) / Get matrix

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 · 2025-10-10T05:01:38Z

❌ Preview Environment deleted from Bunnyshell

Available commands:

🚀 /bns:deploy to redeploy the environment

coderabbitai

Actionable comments posted: 1

🧹 Nitpick comments (2)

.github/workflows/ci__full_2_1.yaml (1)
6-6: Fix the cron comment to match actual schedule.

The cron now runs at 02:00 UTC, but the inline comment still claims 03:00. Please update the comment to avoid confusion.

Apply this diff:
-            cron: "0 2 * * *" # Run every day at 3am
+            cron: "0 2 * * *" # Run every day at 2am
.github/workflows/ci__full_2_2.yaml (1)
10-10: Align the concurrency group suffix with the 2.2 workflow.

The concurrency group still ends with -2_0-full, which can confuse tracking and cancellation logic now that this workflow targets 2.2.

Apply this diff:
-    group: ci-${{ github.workflow }}-${{ github.ref }}-2_0-full
+    group: ci-${{ github.workflow }}-${{ github.ref }}-2_2-full

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7dcdad3 and 7edcd0b.

📒 Files selected for processing (3)

.github/workflows/ci__full_2_1.yaml (1 hunks)
.github/workflows/ci__full_2_2.yaml (6 hunks)
.github/workflows/matrix.json (7 hunks)

.github/workflows/matrix.json

.github/workflows/ci__full_2_1.yaml

GSadee added 2 commits October 10, 2025 06:48

[CI] Replace the full build for 2.0 with the build for 2.2 branch

6992cd4

[CI] Reduce number of builds

7edcd0b

GSadee requested review from a team as code owners October 10, 2025 05:00

GSadee added the Maintenance CI configurations, READMEs, releases, etc. label Oct 10, 2025

coderabbitai bot reviewed Oct 10, 2025

View reviewed changes

.github/workflows/matrix.json Show resolved Hide resolved

Rafikooo reviewed Oct 10, 2025

View reviewed changes

.github/workflows/ci__full_2_1.yaml Outdated Show resolved Hide resolved

[CI] Fix comment and twig version format

c5e740f

Rafikooo approved these changes Oct 13, 2025

View reviewed changes

Rafikooo merged commit 1a6ac44 into Sylius:2.1 Oct 13, 2025
27 checks passed

coderabbitai bot mentioned this pull request Nov 21, 2025

[CI] Add PHP 8.5 support to workflows #18563

Merged

This was referenced Nov 29, 2025

[CI] Split Chromedriver tests into parallel jobs #18603

Merged

Remove 2.0 branch references from CI workflows #18608

Merged

coderabbitai bot mentioned this pull request Jan 16, 2026

[CI] Update Full CI workflows to 1.14, 2.2 and 2.3 #18733

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[CI] Replace the full build for 2.0 with the build for 2.2 branch + reduce number of builds#18431

[CI] Replace the full build for 2.0 with the build for 2.2 branch + reduce number of builds#18431
Rafikooo merged 3 commits intoSylius:2.1from
GSadee:SYL-5612-ci-remove-2.0-full-build

GSadee commented Oct 10, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Oct 10, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Oct 10, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

GSadee commented Oct 10, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

github-actions bot commented Oct 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ Preview Environment deleted from Bunnyshell

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

GSadee commented Oct 10, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Oct 10, 2025 •

edited

Loading

github-actions bot commented Oct 10, 2025 •

edited

Loading