Run Dask's test_pickle_empty on CI again

[jakirkham]

We needed to skip this in PR ( #432 ) as it was failing. This has since been fixed upstream ( dask/distributed#5303 ). So readd testing of it.

[codecov]

Codecov Report

Merging #442 (cc6c1a5) into master (ef6fb90) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #442   +/-   ##
=======================================
  Coverage   92.36%   92.36%           
=======================================
  Files           4        4           
  Lines         720      720           
  Branches      150      150           
=======================================
  Hits          665      665           
  Misses         34       34           
  Partials       21       21           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ef6fb90...cc6c1a5. Read the comment docs.

[pierreglaser]

LGTM, thanks!

I've activated running distributed's test suite for this PR. Can you push an empty commit to trigger a new build?

[jakirkham]

Yep thanks Pierre 😄 Pushed an empty commit above

[jakirkham]

There was one more flaky test. Added that to skip list with a reference. Otherwise seems to be working 🙂

[pierreglaser]

thank you very much @jakirkham for your support on the distributed-side of cloudpickle'CI!

[jakirkham]

Ofc thanks for your help as well, Pierre :)