Skip to content

Add GitHub Actions CI, bump minimum PHP to 7.4 and WP to 6.5#75

Open
joehoyle wants to merge 8 commits intomainfrom
add-github-actions-ci
Open

Add GitHub Actions CI, bump minimum PHP to 7.4 and WP to 6.5#75
joehoyle wants to merge 8 commits intomainfrom
add-github-actions-ci

Conversation

@joehoyle
Copy link
Copy Markdown
Member

Replaces Travis CI with a GitHub Actions workflow that runs PHPUnit against PHP 7.4–8.3 × WordPress 6.5/latest, plus a PHPCS job. Updates PHPUnit to ^9.6 and cleans up the phpunit.xml.dist for PHPUnit 9.

joehoyle and others added 8 commits April 30, 2026 17:37
Replaces Travis CI with a GitHub Actions workflow that runs PHPUnit
against PHP 7.4–8.3 × WordPress 6.5/latest, plus a PHPCS job.
Updates PHPUnit to ^9.6 and cleans up the phpunit.xml.dist for PHPUnit 9.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add yoast/phpunit-polyfills (required by WP 5.9+ test bootstrap)
- Define WP_TESTS_PHPUNIT_POLYFILLS_PATH in test bootstrap
- Fix 3 pre-existing Yoda condition violations in inc/

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…sion

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Tests cover Client CRUD and validation, Authorization_Code lifecycle,
Access_Token creation and revocation for both user and client-credential
tokens, the REST token endpoint (authorization code and client credentials
grants), authentication helpers, and top-level namespace functions.

Excludes tests/ directory from PHPCS since PHPUnit naming conventions
conflict with WordPress file-name rules.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
WordPress 5.5 requires permission_callback on all REST routes.
Both public endpoints get __return_true since they handle their own
authentication internally.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@joehoyle joehoyle requested a review from rmccue May 1, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant