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

Add a magic wrapper to allow phpunit4 to run the code again #2137

Merged
merged 1 commit into from
Nov 15, 2016

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Nov 15, 2016

@mention-bot
Copy link

@nickvergessen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @ChristophWurst, @owncloud-bot and @PVince81 to be potential reviewers.

@nickvergessen nickvergessen force-pushed the make-sure-tests-run-on-phpunit-4-again branch from 4fce534 to 0a4488c Compare November 15, 2016 13:38
nickvergessen added a commit that referenced this pull request Nov 15, 2016
Remove this once phpunit 5 is the lowest supported version, by reverting:
#2137

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen changed the title Need the PR number Add a magic wrapper to allow phpunit4 to run the code again Nov 15, 2016
Remove this once phpunit 5 is the lowest supported version, by reverting:
#2137

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the make-sure-tests-run-on-phpunit-4-again branch from 0a4488c to e76ef9a Compare November 15, 2016 13:50
@nickvergessen
Copy link
Member Author

Fixes the issue: nextcloud/gallery#172

@nickvergessen
Copy link
Member Author

Review please @oparoz @LukasReschke @MorrisJobke

@oparoz
Copy link
Member

oparoz commented Nov 15, 2016

I love magic! 👍

@oparoz
Copy link
Member

oparoz commented Nov 15, 2016

This test PR proves that it fixes things:
nextcloud/gallery#172

@rullzer
Copy link
Member

rullzer commented Nov 15, 2016

Fine by me. As long as we still have apps that need to support <NC11 👍

@rullzer rullzer merged commit 28babd3 into master Nov 15, 2016
@rullzer rullzer deleted the make-sure-tests-run-on-phpunit-4-again branch November 15, 2016 20:48
nickvergessen added a commit that referenced this pull request Nov 24, 2016
Remove this once phpunit 5 is the lowest supported version, by reverting:
#2137

Signed-off-by: Joas Schilling <coding@schilljs.com>
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.

4 participants