Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.91%. Comparing base (88464f6) to head (e133d0b).
Report is 58 commits behind head on main.

@@           Coverage Diff           @@
##             main     #613   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          59       59           
  Lines        2326     2405   +79     
=======================================
+ Hits         2324     2403   +79     
  Misses          2        2           

Should this live in https://github.com/zarr-developers/numcodecs/blob/main/numcodecs/checksum32.py, where we already have CRC32? As I understand (from a stackoverflow post) CRC32 and CRC32C are basically the same algorithm, so it makes sense to put them in the same namespace.

I didn't look for checksum32.py. Thanks for telling me.
I just looked into that. A problem is that the existing checksum codecs prepend the checksum whereas zarr3 expects the checksum to be appended. Either we single out crc32c to append the checksum or we add an option to all checksum codecs. Wdyt?

I moved the code to checksum32.py and refactored the tests (they had no parameterization) in 7e78d93

I don't see why this PR isn't ready to merge (it might be just late)? Do I need another approval?