-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 PHP 7.3 runtime #4182
Add PHP 7.3 runtime #4182
Conversation
Added to swagger. Thanks @rabbah. |
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.
LGTM
docs/actions-php.md
Outdated
- guzzlehttp/guzzle | ||
- ramsey/uuid | ||
|
||
The specific versions of these packages depends on the PHP runtime in use and is listed in the |
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.
The specific versions of these packages depends on the PHP runtime in use and is listed in the | |
The specific versions of these packages depends on the PHP runtime in use and is listed in the |
remove trailing space
Codecov Report
@@ Coverage Diff @@
## master #4182 +/- ##
==========================================
- Coverage 86.03% 81.18% -4.86%
==========================================
Files 152 152
Lines 7292 7292
Branches 472 472
==========================================
- Hits 6274 5920 -354
- Misses 1018 1372 +354
Continue to review full report at Codecov.
|
Add support for the new PHP 7.3 runtime.
Related issue and scope
My changes affect the following components
Types of changes
Checklist: