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

Fix a few bugs introduced by the Python 3 support and add test coverage #21

Merged
merged 1 commit into from
Jun 30, 2015

Conversation

griffy
Copy link
Contributor

@griffy griffy commented Jun 18, 2015

This adds pytest as the testing tool and adds a test command that can be invoked with: python setup.py test

@griffy griffy mentioned this pull request Jun 18, 2015
@fjy
Copy link
Member

fjy commented Jun 23, 2015

👍

@drcrallen
Copy link
Member

@griffy Went through some PRs that were in the backlog, can you please fix conflicts?

@griffy
Copy link
Contributor Author

griffy commented Jun 26, 2015

@drcrallen @fjy Fixed the conflicts.

@xvrl
Copy link
Member

xvrl commented Jun 30, 2015

@griffy do you mind squashing commits, and I'll merge

@griffy
Copy link
Contributor Author

griffy commented Jun 30, 2015

@xvrl Squashed

xvrl added a commit that referenced this pull request Jun 30, 2015
Fix a few bugs introduced by the Python 3 support and add test coverage
@xvrl xvrl merged commit d9978d6 into druid-io:master Jun 30, 2015
@xvrl xvrl modified the milestone: 0.2.2 Jul 28, 2015
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