Skip to content

Require PHP 8.2+, add Pest tests - #31

Merged
retlehs merged 1 commit into
masterfrom
add/pest-tests
Mar 9, 2026
Merged

retlehs merged 1 commit into
masterfrom
add/pest-tests

Conversation

@retlehs

@retlehs retlehs commented Mar 9, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Update minimum PHP requirement from 8.1 to 8.2
  • Widen Laravel Pint constraint to ^1.20 (no longer capped below 1.21)
  • Add Pest (^2.36 || ^3.0 || ^4.0) as a dev dependency
  • Add 7 unit tests for relative_uri() and 1 behavior test for URL assembly
  • Add minimal WP function stubs and WP_Admin_Bar fake for testing without WordPress
  • Add test step to CI workflow, use composer update instead of composer install
  • Drop PHP 8.1 from CI matrix
  • Remove composer.lock from tracking and add to .gitignore
  • Add .gitattributes to exclude dev files from dist archives

🤖 Generated with Claude Code

@retlehs retlehs self-assigned this Mar 9, 2026
@retlehs
retlehs force-pushed the add/pest-tests branch 3 times, most recently from 67ca9c8 to c6091cf Compare March 9, 2026 21:56
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@retlehs retlehs changed the title Add Pest tests for URL translation Require PHP 8.2+, add Pest tests Mar 9, 2026
@retlehs
retlehs merged commit 7cf53dc into master Mar 9, 2026
4 checks passed
@retlehs
retlehs deleted the add/pest-tests branch March 9, 2026 21:58
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