Skip to content

Conversation

@keewis
Copy link
Collaborator

@keewis keewis commented Aug 28, 2024

In the disabled mypy hook we explicitly install types-pkg_resources. This project has been yanked on PyPI, with the notice that instead we should use types-setuptools. This applies that change, but if I remember correctly, we stopped depending on setuptools / pkg_resources a while ago, so we might also be able to just remove that?

@keewis
Copy link
Collaborator Author

keewis commented Aug 28, 2024

since my knowledge on typing is very light, I'll leave that to someone else to pick up. Also, does anyone use the mypy pre-commit hook? Since it's disabled, I tend to check the mypy CI instead.

In any case, I'll go ahead and merge this now, since it unblocks a couple of other PRs.

@keewis keewis merged commit 41d50b0 into pydata:main Aug 28, 2024
@keewis keewis deleted the fix-pre-commit-mypy branch August 28, 2024 18:00
hollymandel pushed a commit to hollymandel/xarray that referenced this pull request Aug 30, 2024
* use `types-setuptools` instead of the deprecated `types-pkg_resources`

* [skip-ci][skip-rtd]
dcherian added a commit to dcherian/xarray that referenced this pull request Aug 30, 2024
* main:
  Accessibility: Add keyboard handling for XArray HTML view (pydata#9412)
  [pre-commit.ci] pre-commit autoupdate (pydata#9316)
  [skip-ci] Speed up docs build by limiting toctrees (pydata#9395)
  fix the failing `pre-commit.ci` runs (pydata#9411)
  Update benchmarks.yml (pydata#9406)
  GroupBy(multiple groupers) (pydata#9372)
  Encode/decode property tests use variables() (pydata#9401)
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.

1 participant