Closed Slamdunk closed 2 years ago
@Slamdunk I would just make sure we improve what we can, without breaking BC. I've done hundreds of these patches on laminas/
: we don't need to refactor anything particular, just the diff lines :)
I'll remove the 7.4 pipeline requirement once ready and releasable :+1:
Released: https://github.com/phptal/PHPTAL/releases/tag/1.7.0
Thanks @Slamdunk!
I admit I've done the minimal effort to get the tests pass because code quality without CSFix & SA tools is a pain.
I'm undecided between answering your change requests with "Ain't nobody got time for that" or "Ok but only with
doctrine/coding-standard
andpsalm
at level 6"