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

Drop unneeded Dask version check from tests #85

Merged
merged 1 commit into from
Oct 9, 2017
Merged

Conversation

jakirkham
Copy link
Owner

This is a holdover from testing some functions included for compatibility with older versions of Dask. Given those functions and their associated tests dropped, there is no longer a need for this check. So drop it as well.

@jakirkham jakirkham changed the title Drop unneeded Dask version check Drop unneeded Dask version check from tests Oct 9, 2017
@jakirkham
Copy link
Owner Author

As a little history, this check was added in PR ( #49 ). It became unnecessary after the changes made in PR ( #71 ).

This is a holdover from testing some functions included for
compatibility with older versions of Dask. Given those functions and
their associated tests dropped, there is no longer a need for this
check. So drop it as well.
@jakirkham jakirkham merged commit 95d6176 into master Oct 9, 2017
@jakirkham jakirkham deleted the drop_dask_ver_chck branch October 9, 2017 02:25
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.

None yet

1 participant