zarr-python icon indicating copy to clipboard operation
zarr-python copied to clipboard

Apply assorted ruff/refurb rules (FURB)

Open DimitriPapadopoulos opened this issue 1 year ago • 1 comments

Note: #1873 is the same applied to the v3 branch

FURB113 Use `compressors.extend(...)` instead of repeatedly calling `compressors.append()`
FURB118 Use `operator.mul` instead of defining a lambda
FURB171 Membership test against single-item container

TODO:

  • [ ] Add unit tests and/or doctests in docstrings
  • [ ] Add docstrings and API docs for any new/modified user-facing classes and functions
  • [ ] New/modified features documented in docs/tutorial.rst
  • [ ] Changes documented in docs/release.rst
  • [x] GitHub Actions have all passed
  • [x] Test coverage is 100% (Codecov passes)

DimitriPapadopoulos avatar Mar 12 '24 18:03 DimitriPapadopoulos

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.98%. Comparing base (cb4230d) to head (24bd82a). Report is 24 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##           main    #1707       +/-   ##
=========================================
+ Coverage      0   99.98%   +99.98%     
=========================================
  Files         0       38       +38     
  Lines         0    14666    +14666     
=========================================
+ Hits          0    14664    +14664     
- Misses        0        2        +2     
Files with missing lines Coverage Δ
zarr/_storage/v3_storage_transformers.py 100.00% <100.00%> (ø)
zarr/meta.py 100.00% <100.00%> (ø)
zarr/tests/test_core.py 99.94% <100.00%> (ø)

... and 35 files with indirect coverage changes

codecov[bot] avatar Mar 12 '24 18:03 codecov[bot]

@DimitriPapadopoulos - thanks for moving your focus over to the v3 branch. I'm going to close this in favor of the contributions you've already made there. 🙏

jhamman avatar Jun 01 '24 16:06 jhamman