Skip to content

Update the CI setup - #65

Merged
uuf6429 merged 1 commit into
minkphp:mainfrom
stof:ci
Oct 7, 2026
Merged

uuf6429 merged 1 commit into
minkphp:mainfrom
stof:ci

Conversation

@stof

@stof stof commented Nov 23, 2025 •

Copy link
Copy Markdown
Member
  • add CI jobs using PHP 8.4 and 8.5
  • use PHP 8.5 as default PHP version
  • update actions used in workflows

@codecov

codecov Bot commented Nov 23, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.06%. Comparing base (acd45d2) to head (75454ed).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #65   +/-   ##
=========================================
  Coverage     89.06%   89.06%           
  Complexity      197      197           
=========================================
  Files             1        1           
  Lines           494      494           
=========================================
  Hits            440      440           
  Misses           54       54           
Flag Coverage Δ
chrome 88.46% <ø> (ø)
chromium 87.85% <ø> (ø)
edge 88.25% <ø> (ø)
firefox 88.05% <ø> (ø)
selenium-2.53.1 87.04% <ø> (ø)
selenium-3 88.05% <ø> (ø)
selenium-4 88.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- add CI jobs using PHP 8.4 and 8.5
- use PHP 8.5 as default PHP version
- update actions used in workflows

@aik099 aik099 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Optionally (to make CI more green) you could rebase on main to have selenium server 4 + phpstan checks pass.

@uuf6429
uuf6429 merged commit efa6de0 into minkphp:main Oct 7, 2026
14 of 19 checks passed
@uuf6429

uuf6429 commented Oct 7, 2026

Copy link
Copy Markdown
Member

Looks good to me - the other failures are definitely unrelated and some couldn't be avoided anyway. Merged.

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.

3 participants