Skip to content

Conversation

max-sixty
Copy link
Collaborator

Summary

This PR fixes mypy type checking errors in test_backends.py that were causing CI failures on the main branch.

Changes

  • Added explicit keyword arguments (varname=, datatype=, dimensions=) to netCDF4.Dataset.createVariable calls
  • Added # type: ignore[call-overload] comments where the dynamic compression parameter couldn't be statically verified
  • Removed an unnecessary # type: ignore[import-untyped] for the requests import (type stubs are now available)

Test plan

  • Verified no mypy errors remain in test_backends.py
  • All pre-commit checks pass
  • The modified _check_compression_codec_available function still works correctly
  • Module imports without errors

🤖 Generated with Claude Code

- Add explicit keyword arguments to netCDF4.Dataset.createVariable calls
- Add type: ignore comments for dynamic compression parameter
- Remove unnecessary type: ignore for requests import

These changes resolve mypy type checking errors that were causing CI failures.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@max-sixty max-sixty enabled auto-merge (squash) September 3, 2025 21:33
@max-sixty max-sixty merged commit 1d47cbf into pydata:main Sep 3, 2025
36 checks passed
@max-sixty max-sixty deleted the mypy branch September 3, 2025 21:38
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