Repository navigation
ci: add php-ai-client PHPUnit matrix - #39
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Co-authored-by: Darin Kotter <darin.kotter@gmail.com>
|
Addressed the review feedback:
WordPress-version combinations are not needed because the unit suite does not bootstrap WordPress; compatibility with WordPress 7.0's bundled client is represented by the 1.3.1 dependency floor. All 12 PHP/dependency matrix jobs and the existing CI jobs pass. |
âĶ into ci/php-ai-client-phpunit-matrix
|
I'm seeing double tests run on this PR and in looking into why, seems we now have another testing workflow merged as part of #40. So this PR may not be needed or we may want to look at the workflow added in that PR and make any adjustments that have come out of discussion on this PR. |
|
@superdav42 can you adjust the work in this PR so that its focused on iterating on https://github.com/WordPress/ai-provider-for-openai/blob/trunk/.github/workflows/ci.yml and removing the specific |
|
@jeffpaul Adjusted:
|
Summary
Verification
composer update wordpress/php-ai-client --with=wordpress/php-ai-client:1.3.1 --with-all-dependencies --no-interaction --no-progress --prefer-dist --dry-runcomposer update wordpress/php-ai-client --with=wordpress/php-ai-client:1.4.0 --with-all-dependencies --no-interaction --no-progress --prefer-dist --dry-runcomposer testâ 9 tests, 29 assertions with php-ai-client 1.4.0For #34
aidevops.sh v3.32.126 plugin for OpenCode v1.18.2