-
Notifications
You must be signed in to change notification settings - Fork 31
Add phpunit 11 support #86
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
Conversation
@pieterocp Just a thought: if you literally copy someone else's code .... you may want to give them credit...
Credited to dmaicher Co-authored-by: dmaicher <dmaicher@users.noreply.github.com>
... this version works with most/all phpunit installations Credit to jrfnl for the suggestion and the phar test setup from Yoast Co-authored-by: jrfnl <jrfnl@users.noreply.github.com>
ec46deb to
6b1adb8
Compare
@pieterocp Just a thought: if you literally copy someone else's code .... you may want to give them credit...
Apologies, added to the commit messages the folks who's work I glued together. So thank you @jrfnl and @dmaicher.
@pieterocp Appreciated.
dbarneveld
commented
Mar 12, 2025
@rdohms could you review this PR?
Lukasss93
commented
Mar 26, 2025
any news about this?
matthiasPOE
commented
Oct 2, 2025
@rdohms can you please take a look at this or give the repo to another maintainer?
Uh oh!
There was an error while loading. Please reload this page.
Allow for phpunit version 11.* to be used, addressing #85 + should handle the case of https://github.com/rdohms/phpunit-arraysubset-asserts/pull/81/files#r1557304228 so thank you for the suggestion.
Did run the job using
nektos/actand outside of being ratelimited if I ran several at once, it seemed to generally work. The version constraints can probably be changed a bit though.Tested with inside a project, which has this composer installed, plus ran tests locally on using a phpunit.phar installation.
Closes #83, #85, #87