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

Temp Fix - PHPUnit 9 #308

Merged
merged 1 commit into from
May 5, 2023
Merged

Temp Fix - PHPUnit 9 #308

merged 1 commit into from
May 5, 2023

Conversation

J-Brk
Copy link
Collaborator

@J-Brk J-Brk commented Apr 28, 2023

Pin PHPUnit to version 9; to make sure the automated tests remain operational.

Q A
Is bugfix? ✔️
New feature?
Breaks BC?
Tests pass? ✔️
Fixed issues

@J-Brk
Copy link
Collaborator Author

J-Brk commented Apr 28, 2023

@vinicius73 Can you review this request primarily? I have added it because PHPUnit updated to version 10; that makes all our automated checks fail since they are based on version 9. This merge pins PHPUnit to version 9 for now; while I'm preparing the merge for version 10 in #306 . After this v9 has been fixed; #304 and #297 should be able to be merged so please review if they can be done after this merge.

@J-Brk
Copy link
Collaborator Author

J-Brk commented May 5, 2023

@vinicius73 Normally I wait for you to do the final review of the PR before merging but since this is a very minor adjustment & to prevent delay for item #304; I have decided to push ahead and merge this item.

@J-Brk J-Brk merged commit 5358a7d into master May 5, 2023
@J-Brk J-Brk deleted the dev-temp-phpunit-fix branch May 5, 2023 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant