Skip `checksum32` tests if `crc32c` is missing by jakirkham · Pull Request #635 · zarr-developers/numcodecs · GitHub
Skip to content

Skip checksum32 tests if crc32c is missing#635

Merged
jakirkham merged 1 commit into
zarr-developers:mainfrom
jakirkham:skip_crc32_tsts
Nov 13, 2024
Merged

Skip checksum32 tests if crc32c is missing#635
jakirkham merged 1 commit into
zarr-developers:mainfrom
jakirkham:skip_crc32_tsts

Conversation

@jakirkham

@jakirkham jakirkham commented Nov 13, 2024

Copy link
Copy Markdown
Member

If crc32c is not installed, skip the tests in test_checksum32.py.


TODO:

  • Unit tests and/or doctests in docstrings
  • Tests pass locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • Docs build locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Codecov passes)

@jakirkham jakirkham changed the title Skip checksum32 tests if crc32 is missing Skip checksum32 tests if crc32c is missing Nov 13, 2024
@codecov

codecov Bot commented Nov 13, 2024

Copy link
Copy Markdown

@jakirkham jakirkham merged commit 29995e3 into zarr-developers:main Nov 13, 2024
@jakirkham jakirkham deleted the skip_crc32_tsts branch November 13, 2024 19:04
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.

2 participants