Skip to content

fix #4278#4280

Merged
mrocklin merged 3 commits intodask:masterfrom
515hikaru:master
Dec 9, 2018
Merged

fix #4278#4280
mrocklin merged 3 commits intodask:masterfrom
515hikaru:master

Conversation

@515hikaru
Copy link
Contributor

@515hikaru 515hikaru commented Dec 8, 2018

I made it possible to run a test with pytest==4.0.1(this version is latest now.)

related issue: #4278

  • Tests added / passed
  • Passes flake8 dask

@515hikaru 515hikaru changed the title fix #4278 WIPfix #4278 Dec 8, 2018
@515hikaru 515hikaru changed the title WIPfix #4278 WIP: fix #4278 Dec 8, 2018
@mrocklin
Copy link
Member

mrocklin commented Dec 8, 2018

Thank you for starting this @515hikaru . Are you interested in doing this same work for the other tests?

You can get a list of all failing tests with by looking at the "Warnings Summary" in one of the Travis CI logs. You could also change the pytest version here:

https://github.com/dask/dask/blob/master/continuous_integration/travis/install.sh#L54

@515hikaru
Copy link
Contributor Author

@mrocklin: Thank you for your information. I want to fix other tests.

I will change pytest version when all tests fixed.

* dask/bytes
* dask/dataframe/io/tests

and change pytest version
@515hikaru 515hikaru changed the title WIP: fix #4278 fix #4278 Dec 9, 2018
@515hikaru
Copy link
Contributor Author

515hikaru commented Dec 9, 2018

I fixed all tests, and changed pytest version in travis job.

Although travis 3rd job fails, I do not know how to fix it.It seems this issue is another one, is not it?

@mrocklin
Copy link
Member

mrocklin commented Dec 9, 2018

Yes, it's unrelated and being handled in #4276 . Merging this now. Thank you @515hikaru !

If you want to continue with this work the other Dask repositories like dask/distributed and dask/dask-ml probably also have this same problem.

I also notice that this is your first code contribution to this repository. Welcome!

@mrocklin mrocklin merged commit 8bd87a4 into dask:master Dec 9, 2018
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.

2 participants