pre-commit: spell, whitespace, and mypy check

[madsbk]

close #1077

[codecov-commenter]

Codecov Report

Base: 87.90% // Head: 87.90% // No change to project coverage 👍

Coverage data is based on head (072d5d4) compared to base (4f0922c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@              Coverage Diff              @@
##           branch-23.02    #1091   +/-   ##
=============================================
  Coverage         87.90%   87.90%           
=============================================
  Files                17       17           
  Lines              2298     2298           
=============================================
  Hits               2020     2020           
  Misses              278      278           

Impacted Files	Coverage Δ
dask_cuda/cli.py	86.40% <ø> (ø)
dask_cuda/disk_io.py	56.19% <ø> (ø)
dask_cuda/explicit_comms/dataframe/shuffle.py	98.67% <ø> (ø)
dask_cuda/initialize.py	87.23% <ø> (ø)
dask_cuda/is_spillable_object.py	84.37% <ø> (ø)
dask_cuda/proxy_object.py	90.52% <ø> (ø)
dask_cuda/utils.py	83.47% <ø> (ø)
dask_cuda/proxify_device_objects.py	97.32% <100.00%> (ø)
dask_cuda/proxify_host_file.py	93.71% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[pentschev]

LGTM, thanks @madsbk . What's the idea moving forward, make mypy strict?

[madsbk]

LGTM, thanks @madsbk . What's the idea moving forward, make mypy strict?

I think the first step is to install all the library stubs in CI. Currently, a lot of the code is ignored.

[wence-]

LGTM, thanks @madsbk . What's the idea moving forward, make mypy strict?

I think the first step is to install all the library stubs in CI. Currently, a lot of the code is ignored.

If we're using pre-commit, then we need to specify the extra packages in the pre-commit environment, so they all need to be pip installable (which might be problematic?)

[madsbk]

/merge