-
Notifications
You must be signed in to change notification settings - Fork 38
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
Set up automated testing #640
Comments
@danieliser I did some research on this and we can definitely utilize GitHub Actions for this since our tests are rather straightforward. The question is: When should we run the tests? Just pushes on |
@fpcorso I think this ideally should occur when a PR is submitted against Probably same for linting and other checks as well. |
@danieliser I see two problems doing it only for PR's for
This is why I was thinking we should also do PR's to |
@fpcorso My only concern with doing it off So I guess all merges into |
Creates a new workflow that will validate composer, install dependencies, and run PHPUnit tests. Issue #640
Now that we have PHPUnit set up and some integration tests created, we need these to be automatically run during pull requests, commits, merges, etc...
Merge these into something usable.
The text was updated successfully, but these errors were encountered: