New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GH Actions: run tests against PHP 8.1 and other tweaks #172
GH Actions: run tests against PHP 8.1 and other tweaks #172
Commits on Aug 8, 2021
-
GH Actions: ubuntu-16.04 is no longer supported
... use `ubuntu-18.04` or `ubuntu-latest` for `20.04` instead. Also see: * https://ubuntu.com/blog/ubuntu-16-04-lts-transitions-to-extended-security-maintenance-esm * shivammathur/setup-php#452
Configuration menu - View commit details
-
Copy full SHA for 76d27d0 - Browse repository at this point
Copy the full SHA 76d27d0View commit details -
GH Actions: explicitly set code coverage to none
As no code coverage is being recorded for these builds, it is good practice to explicitly set `coverage: none` in `setup-php`. This fixes a warning on PHP 5.3 stating that Xdebug is on.
Configuration menu - View commit details
-
Copy full SHA for e386675 - Browse repository at this point
Copy the full SHA e386675View commit details -
GH Actions: enable Composer caching
... by using the `ramsey/composer-install` action. This means that the Composer downloads directory for dependencies will be cached and restored on each build. This conserves resources and should also make builds faster. Ref: https://github.com/marketplace/actions/install-composer-dependencies
Configuration menu - View commit details
-
Copy full SHA for 6ab88bd - Browse repository at this point
Copy the full SHA 6ab88bdView commit details -
GH Actions: split off Psalm to separate workflow
Psalm does not need to be run against multiple PHP versions. Running it once should be enough. With that in mind, this commit: * Introduces a separate, dedicated workflow which only installs and runs Psalm. * Removes the Psalm related steps from the `CI` workflow. * Removes Psalm from the `tools` setting for `setup-php`.
Configuration menu - View commit details
-
Copy full SHA for 64256a0 - Browse repository at this point
Copy the full SHA 64256a0View commit details -
GH Actions: clean up running of the tests
In contrast to Sodium Compat, the tests for this package _were_ running on the low PHP versions. However, the point remains that PHPUnit was being installed 3 (!) times, once via the `setup-php` action, once via the `composer install` and once via the `php-actions/phpunit` action. This simplifies the script and ensures that the tests are always run against the most appropriate PHPUnit version for the PHP version against which the tests are being run, by: * Remove the installing of PHPUnit via `setup-php`. * Remove the use of the `php-actions/phpunit` action. * Defer to the Composer installed PHPUnit version in all cases. I'm also removing the explicit ini settings for the jobs. These look like they were copied over from example code, but these values don't have any effect on the test runs in these workflows, so may as well be removed.
Configuration menu - View commit details
-
Copy full SHA for d0bbad2 - Browse repository at this point
Copy the full SHA d0bbad2View commit details -
GH Actions: merge "moderate" and "modern" jobs
As there is now effectively no difference anymore between the `moderate` and `modern` jobs, these jobs can now be merged into one.
Configuration menu - View commit details
-
Copy full SHA for 71c5eb7 - Browse repository at this point
Copy the full SHA 71c5eb7View commit details -
GH Actions: set error reporting to E_ALL
The default setting for `error_reporting` used by the SetupPHP action is `error_reporting=E_ALL & ~E_DEPRECATED & ~E_STRICT` and `display_errors` is set to `Off`. For the purposes of CI, I'd recommend running with `E_ALL` and `display_errors=On` to ensure **all** PHP notices are shown. Ref: shivammathur/setup-php#469
Configuration menu - View commit details
-
Copy full SHA for c84f7b7 - Browse repository at this point
Copy the full SHA c84f7b7View commit details -
GH Actions: enable testing against PHP 8.1
For now, this build is still allowed to fail.
Configuration menu - View commit details
-
Copy full SHA for 8100036 - Browse repository at this point
Copy the full SHA 8100036View commit details