-
Notifications
You must be signed in to change notification settings - Fork 173
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.
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
Add support for php8.3 #3642
Add support for php8.3 #3642
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3642 +/- ##
==========================================
+ Coverage 1.81% 2.07% +0.26%
==========================================
Files 113 89 -24
Lines 6130 5348 -782
Branches 1491 1491
==========================================
Hits 111 111
+ Misses 5901 5119 -782
Partials 118 118 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No CI coverage
a6fd7bf
to
a682f13
Compare
.github/workflows/phpunit.yml
Outdated
@@ -88,7 +88,7 @@ jobs: | |||
# do not stop on another job's failure | |||
fail-fast: false | |||
matrix: | |||
php-versions: ['8.0', '8.1', '8.2'] | |||
php-versions: ['8.0', '8.1', '8.2', '8.3'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
works, but bloats the matrix. could you move 8.0 out of this array and have one dedicated job in the include
section below?
Signed-off-by: hamza221 <hamzamahjoubi221@gmail.com>
a682f13
to
91d2cb8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😎
closes #3641