Skip to content
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

Update dependencies and Dockerfile versions for CI #27

Merged
merged 6 commits into from
Apr 25, 2024

Conversation

missionmike
Copy link
Contributor

Description

This PR updates a few dependencies:

  • PHP version in composer.json's require to minimum 8.1 to match the workflow matrix minimum.
  • lucatime/wpbrowser to latest.
  • PHP version in Dockerfile to 8.1
  • XDebug version in Dockerfile to 3.3.1 to work with PHP 8.1 (see https://xdebug.org/docs/compat)

After updating the dependencies and Dockerfile, a few values needed to be set:

  • ENV for XDebug coverage in Dockerfile
  • WP values in local/config/wp-config.php file for Codeception. Without these values, there were errors and the run-codeception script would fail.

Testing

To test locally, checkout this branch, then delete the following files/folders from local:

image

Then, run:

  • composer install
  • docker-compose up -d --build --force-recreate
  • composer run-codeception -- -- --coverage --coverage-xml
  • composer run-phpunit -- -- --coverage-text

All should run successfully.

Example of passing CI: https://github.com/missionmike/wp-graphql-testcase/actions/runs/8649079504

@kidunot89 kidunot89 self-requested a review April 25, 2024 19:48
@kidunot89 kidunot89 merged commit 91dbafc into wp-graphql:develop Apr 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants