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

[TASK] Configure PHPUnit some more #436

Merged
merged 1 commit into from
Feb 4, 2024
Merged

Conversation

oliverklee
Copy link
Contributor

  • make things more strict
  • stop caching the result (as we don't need this at the moment)
  • make things more colorful

- make things more strict
- stop caching the result (as we don't need this at the moment)
- make things more colorful
@oliverklee oliverklee force-pushed the task/phpunit-configuration branch from a1ff750 to 4854326 Compare February 3, 2024 21:08
Copy link
Contributor

@JakeQZ JakeQZ left a comment

Choose a reason for hiding this comment

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

It could be argued that all of these settings should be the default.

@JakeQZ JakeQZ merged commit 084e376 into master Feb 4, 2024
26 checks passed
@JakeQZ JakeQZ deleted the task/phpunit-configuration branch February 4, 2024 01:27
@oliverklee
Copy link
Contributor Author

It could be argued that all of these settings should be the default.

Sebastian (the author of PHPUnit) actually sometimes actually does this for newer versions of PHPUnit. :-)

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