search

dxw / whippet

Whippet is a framework for building WordPress sites that eases deployment, plugin management and build steps.
MIT License
14 stars 3 forks source link

Bugs in PHP 8.2/Increase test coverage #221

Open RobjS opened 1 year ago

RobjS commented 1 year ago

A number of classes (e.g. the generators) have no test coverage. As a result, there are some commands that will throw errors in PHP 8.2 (e.g. whippet generate theme) as the errors were not surfaced by the test suite when we did the initial PHP 8.2 work. We should ensure we have test coverage (at least unit, if not integration) for all the codebase, to avoid this in future.

snim2 commented 1 year ago

This might also be an opportunity to remove some repeated code in the generators and extract it into a superclass

snim2 commented 1 year ago

Related: