Skip to content

[FEATURE] Add Composer scripts to run unit tests #503

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

Merged
merged 2 commits into from
Feb 27, 2024

Conversation

JakeQZ
Copy link
Collaborator

@JakeQZ JakeQZ commented Feb 26, 2024

This is the change to composer.json which was inadvertently omitted in #488.

The `ci` script now also runs `ci:dynamic` which in turn runs `ci:tests`.

The PHPUnit cache is only applicable if the configuration is to run previously
failing tests first, which it isn't, hence `--do-not-cache-result`.

Also update the readme instructions for running all code checks.

Relates to #486.
This is the change to `composer.json` which was inadvertently omitted from #488.
@JakeQZ JakeQZ added enhancement developer-specific Issues that only affect maintainers, contributors, and people submitting PRs labels Feb 26, 2024
@JakeQZ JakeQZ self-assigned this Feb 26, 2024
@JakeQZ
Copy link
Collaborator Author

JakeQZ commented Feb 26, 2024

This is the change to composer.json which was inadvertently omitted in #488.

It also includes the same change to README.md that was committed in that PR. I should have added two files to the commit, but managed to only add one of them. Doesn't seem to need a rebase as the change to the file that was committed is identical.

@oliverklee oliverklee merged commit 3a461e0 into main Feb 27, 2024
@oliverklee oliverklee deleted the feature/composer-dynamic branch February 27, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer-specific Issues that only affect maintainers, contributors, and people submitting PRs documentation enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants