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

Added PHPUnit as dev dependency, and support versions 5 and 6. #28

Merged
merged 2 commits into from
Nov 13, 2017
Merged

Added PHPUnit as dev dependency, and support versions 5 and 6. #28

merged 2 commits into from
Nov 13, 2017

Conversation

carusogabriel
Copy link
Contributor

No description provided.

@clue clue added this to the v0.5.1 milestone Nov 13, 2017
@clue
Copy link
Owner

clue commented Nov 13, 2017

Thank you for the PR! 👍 This repo currently doesn't have any test instructions, so now that we add PHPUnit, we should probably document this in our README and use this in Travis accordingly: clue/reactphp-redis#53. Can you update this PR? 👍

Copy link
Owner

@clue clue left a comment

Choose a reason for hiding this comment

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

Thanks for the quick update, LGTM! 🚢 👍

@clue clue merged commit efee12a into clue:master Nov 13, 2017
@carusogabriel carusogabriel deleted the phpunit branch November 14, 2017 09:55
@clue clue modified the milestones: v0.5.1, v0.6.0 Nov 16, 2017
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.

None yet

2 participants