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

Add code coverage and test run via GitHub Action #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

alexander-nitsche
Copy link

Hi Karsten,

i am coming from another project and did some back imports. Mainly automated testing via GitHub action and code coverage configuration. The action proofed to run fine. Should not have any impact on productive use of the package. Leaving it here for merging at any time ..

Greetings
Alex

- Add code coverage configuration to phpunit.xml (100% files, 91% lines).
- Fix: Use name of inheriting test class as test directory name.
- Remember the calculated test directory path to save costs.
@alexander-nitsche
Copy link
Author

Hi Karsten,

did another back import from another project. And prefixed all commit messages with "TASK: " - hence the rebase.

Greetings
Alex

…tion

- Reduce the PHP test matrix to the latest versions of 7 and 8.
  This should be sufficient to find all version-dependent errors.
- Cache Composer dependencies for reuse in different jobs to speed up
  workflows and save bandwidth.

Mind the global warming.
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.

1 participant