Skip to content

ci: update PHP versions in test matrix and simplify test execution #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Mar 31, 2025
Merged

ci: update PHP versions in test matrix and simplify test execution #189

merged 3 commits into from
Mar 31, 2025

Conversation

SRWieZ
Copy link
Member

@SRWieZ SRWieZ commented Mar 29, 2025 •
edited by simonhamp
Loading

  • Removed PHP 8.1 from the matrix of PHP versions.
  • Simplified the test execution step by removing the exclusion of the feature test group.

@SRWieZ
Copy link
Member Author

SRWieZ commented Mar 29, 2025 •
edited
Loading

See there: https://github.com/NativePHP/electron/pull/189/checks?sha=9704f9bdd6e5a28a47759ec78d9059eff259dc6e

https://github.com/NativePHP/electron/actions/runs/14147391745

@simonhamp
Copy link
Member

Why remove 8.1?

@SRWieZ
Copy link
Member Author

SRWieZ commented Mar 31, 2025

I reverted the 8.1 change. I'll do it in another PRs on the three repos.

@SRWieZ SRWieZ merged commit 01270ec into main Mar 31, 2025
57 of 58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simonhamp simonhamp simonhamp approved these changes

@PeteBishwhip PeteBishwhip PeteBishwhip approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SRWieZ @simonhamp @PeteBishwhip