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

fix the failing pre-commit.ci runs #9411

Merged
merged 2 commits into from
Aug 28, 2024
Merged

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
19 checks passed
@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