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 tests for #152 #262

Merged
merged 4 commits into from Oct 28, 2015
Merged

Add tests for #152 #262

merged 4 commits into from Oct 28, 2015

Conversation

@youknowone
Copy link
Contributor

@youknowone youknowone commented Oct 28, 2015

  • Add wand.image.Image.quantize method using MagickQuantizeImage.
  • Add wand.image.Image.transform_colorspace method using MagickTransformImageColorspace
@dahlia
Copy link
Collaborator

@dahlia dahlia commented Oct 28, 2015

Could you adjust docstring formats? They seems slightly inconsistent to existing docstring formats.

@dahlia
Copy link
Collaborator

@dahlia dahlia commented Oct 28, 2015

Plus, it would be better if you write a changelog about this as well.

@youknowone
Copy link
Contributor Author

@youknowone youknowone commented Oct 28, 2015

revised

dahlia added a commit that referenced this pull request Oct 28, 2015
@dahlia dahlia merged commit 5f34dd1 into emcconville:master Oct 28, 2015
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
coverage/coveralls Coverage increased (+0.04%) to 85.416%
Details
@dahlia dahlia mentioned this pull request Oct 28, 2015
@dahlia
Copy link
Collaborator

@dahlia dahlia commented Oct 28, 2015

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants