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

Make rayon an optional dependency #724

Merged
merged 3 commits into from Apr 15, 2018

Conversation

Projects
None yet
2 participants
@Diggsey
Contributor

Diggsey commented Feb 8, 2018

This is a backwards compatible change, but may cause a drop in performance for anyone using --no-default-features. However, I do consider it to be an important change, because a dependency on rayon limits the targets on which image can be used.

@aschampion aschampion referenced this pull request Apr 2, 2018

Closed

New release #743

@bvssvni

This comment has been minimized.

Member

bvssvni commented Apr 15, 2018

Merging.

@bvssvni bvssvni merged commit b650ce6 into PistonDevelopers:master Apr 15, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@bvssvni

This comment has been minimized.

Member

bvssvni commented Apr 15, 2018

Thanks!

@Diggsey Diggsey deleted the Diggsey:optional-rayon branch Jun 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment