Skip to content

Conversation

@Hug0-Drelon
Copy link
Member

What?

  • Add some comment to explain why the need of a "fake" vendor directory for PHPUnit.
  • Erase copied file in the fake vendor directory on each test run (see bootstrap.php).
  • Migrate the CI to GitHub Actions.

… vendor directory and erase vendor/wpsyntex/polylang-phpstan on each test run.
@Hug0-Drelon Hug0-Drelon self-assigned this Jul 17, 2025
@Hug0-Drelon Hug0-Drelon added the enhancement New feature or request label Jul 17, 2025
runs-on: ubuntu-latest
strategy:
matrix:
php-version: [7.4, 8.4]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are there only two versions?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the min and max version supported by our szepeviktor/phpstan-wordpress dependency.
No need to test all versions in between IMHO.

@Hug0-Drelon Hug0-Drelon requested a review from Marianne380 August 4, 2025 15:29
@Chouby Chouby merged commit 0bdb3f2 into master Aug 5, 2025
2 checks passed
@Chouby Chouby deleted the improve-phpunit-tests-and-gh-actions branch August 5, 2025 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants