- tests/bootstrap-unit.php: stop stubbing validateRequestVars() globally; monitor_controller.php declares it unconditionally, so the stub caused a Cannot redeclare fatal once a test loaded plugin source via loadPluginSource()/monitor_test_load().
- .github/workflows/plugin-ci-workflow.yml: add coverage: pcov to the setup-php step so the Pest --coverage-clover report has a driver to run against.
- Document the PHP 8.2 floor (matrix is already 8.2-8.4) in copilot-instructions.md, the php-developer agent, and the hexagonal-monitor ADR, replacing stale PHP 8.1 references now that PHP 8.1 is no longer tested.
- Verified apache2/libapache2-mod-php do not need to be re-added to the Install System Dependencies step: apache2 ships preinstalled on the ubuntu-latest runner image and shivammathur/setup-php configures the matching Apache PHP module for the selected matrix version.
* docs: add Monitor architecture decision record
* ci: add code-quality workflow (lint/cs-fixer)
Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
* docs: keep ADR PR documentation-only
Drop code-quality workflow and php-cs-fixer scaffolding that would
gate merges without a format pass.
Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
* docs: state the PHP 8.1 baseline the CI actually tests
Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
---------
Signed-off-by: Thomas Vincent <thomasvincent@gmail.com>
TheWitness