-
Notifications
You must be signed in to change notification settings - Fork 212
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
ci(SDK-4666): Add support for forthcoming PHP 8.3 release #733
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #733 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 1327 1327
===========================================
Files 62 62
Lines 4613 4613
===========================================
Hits 4613 4613
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -107,7 +107,10 @@ | |||
"pest:fast": [ | |||
"@pest --parallel" | |||
], | |||
"phpcs": "@php vendor/bin/php-cs-fixer fix --dry-run --diff", | |||
"phpcs": [ | |||
"@putenv PHP_CS_FIXER_IGNORE_ENV=1", |
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.
Temporary workaround for PHPCS to run in the pre-release runtime environment successfully.
### Added - Support `initiate_login_uri` property for `PATCH` requests to the `/api/v2/clients/:id` Management API endpoint. [#732](#732) ### Fixed - Token generator typo in claim header. [#729](#729) ### Changed - `composer.json` now identifies `mbstring` as a required extension. [#730](#730) - PHP 8.3 nightlies have been added to the continuous integration testing matrix. [#733](#733)
Changes
This pull request adds the forthcoming PHP 8.3 runtime to the continuous integration test matrix.
References
N/A
Testing
This pull request expands the existing continuous integration testing.
Contributor Checklist