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

mypy - Remove some ignored packages and modules #7319

Merged
merged 8 commits into from
Nov 26, 2022

Conversation

Illviljan
Copy link
Contributor

@Illviljan Illviljan commented Nov 24, 2022

dask has added py.typed files so now the ignores shouldn't be needed anymore:
dask/dask#8854
dask/distributed#5328
As does numpy and pint

pycompat.py doesn't error anymore, so it's good to type check that one as well. Fixed also a python 3.8 related error in it.

@headtr1ck
Copy link
Collaborator

Is numpy not also typed in the meantime?

@headtr1ck
Copy link
Collaborator

The failing doctest is unrelated, or?

@Illviljan
Copy link
Contributor Author

Yeah it must be unrelated, I can't see how the doctest could be related to this.

@Illviljan Illviljan changed the title Allow type checking dask Remove some packages from tool.mypy.overrides Nov 26, 2022
@Illviljan Illviljan changed the title Remove some packages from tool.mypy.overrides mypy - Remove some ignored packages and modules Nov 26, 2022
@Illviljan Illviljan marked this pull request as ready for review November 26, 2022 14:36
@Illviljan Illviljan added the plan to merge Final call for comments label Nov 26, 2022
@headtr1ck
Copy link
Collaborator

I think pandas still requires a stub package. Might still take some time :)

@Illviljan Illviljan merged commit 8953df7 into pydata:main Nov 26, 2022
dcherian added a commit to dcherian/xarray that referenced this pull request Dec 2, 2022
* upstream/main: (39 commits)
  Support the new compression argument in netCDF4 > 1.6.0 (pydata#6981)
  Remove setuptools-scm-git-archive, require setuptools-scm>=7 (pydata#7253)
  Fix mypy failures (pydata#7343)
  Docs: add example of writing and reading groups to netcdf (pydata#7338)
  Reset file pointer to 0 when reading file stream (pydata#7304)
  Enable mypy warn unused ignores (pydata#7335)
  Optimize some copying (pydata#7209)
  Add parse_dims func (pydata#7051)
  Fix coordinate attr handling in `xr.where(..., keep_attrs=True)` (pydata#7229)
  Remove code used to support h5py<2.10.0 (pydata#7334)
  [pre-commit.ci] pre-commit autoupdate (pydata#7330)
  Fix PR number in what’s new (pydata#7331)
  Enable `origin` and `offset` arguments in `resample` (pydata#7284)
  fix doctests: supress urllib3 warning (pydata#7326)
  fix flake8 config (pydata#7321)
  implement Zarr v3 spec support (pydata#6475)
  Fix polyval overloads (pydata#7315)
  deprecate pynio backend (pydata#7301)
  mypy - Remove some ignored packages and modules (pydata#7319)
  Switch to T_DataArray in .coords (pydata#7285)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plan to merge Final call for comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants