Fix CI setup for PHP 8, CakePHP 5.1. Will fix the deprecations in the TestCases in a separate PR.
Fix for the PHPUnit testing. Fixes
PHP Deprecated: Since 5.1.0: Cache config `_cake_core_` is deprecated. Use `_cake_translations_` instead
/home/runner/work/users/users/vendor/cakephp/cakephp/src/I18n/I18n.php, line: 152
You can disable all deprecation warnings by setting `Error.errorLevel` to `E_ALL & ~E_USER_DEPRECATED`. Adding `vendor/cakephp/cakephp/src/I18n/I18n.php` to `Error.ignoredDeprecationPaths` in your `config/app.php` config will mute deprecations from that file only. in /home/runner/work/users/users/vendor/cakephp/cakephp/src/Core/functions.php on line 384
Deprecated: Since 5.1.0: Cache config `_cake_core_` is deprecated. Use `_cake_translations_` instead
/home/runner/work/users/users/vendor/cakephp/cakephp/src/I18n/I18n.php, line: 152
You can disable all deprecation warnings by setting `Error.errorLevel` to `E_ALL & ~E_USER_DEPRECATED`. Adding `vendor/cakephp/cakephp/src/I18n/I18n.php` to `Error.ignoredDeprecationPaths` in your `config/app.php` config will mute deprecations from that file only. in /home/runner/work/users/users/vendor/cakephp/cakephp/src/Core/functions.php on line 384
PHPUnit 10.5.38 by Sebastian Bergmann and contributors.
@umer936 Thanks for the contribution! ...I see the tests are failing..could you please take a look?
@umer936 If you are not available to take a look at this, please let us know so we can check if someone else is available. Thank you!
Oh yeah someone else can go for it. I was just doing the deprecation fix part since it was easy. As the title says, this PR wasn't meant to fix any of the tests anyways, just fix the deprecations.
To me it would make sense to merge this, then do the test fixes as at least it would get past the CI setup part.
But yeah feel free to do whatever - I just like having the deprecations/warnings/etc out of the logs; doesn't matter who gets to it first.