Brain-WP / Cortex

Routing system for WordPress
MIT License
348 stars 20 forks source link

PHP 8.1 QA (Follow-Up) #41

Closed tfrommen closed 1 year ago

tfrommen commented 1 year ago

This PR is a follow-up to #40 that updates the tests to use static::assert* instead of global namespace functions.

Some tests were also missing a mock (e.g., is_ssl or home_url), or setting up the HTTP_HOST server value.

The PR also adds a basic GitHub Action workflow for running tests and PHP compatibility checks (see here).

Feel free to discard this PR, in case you don't want to update all the tests etc., or if you want to do it differently.