Unused import

[DimitriPapadopoulos]

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)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.91%. Comparing base (dbc2975) to head (41b4f6c).
Report is 68 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #580      +/-   ##
==========================================
- Coverage   99.91%   99.91%   -0.01%     
==========================================
  Files          59       59              
  Lines        2334     2332       -2     
==========================================
- Hits         2332     2330       -2     
  Misses          2        2              

Files with missing lines	Coverage Δ
numcodecs/compat.py	100.00% <ø> (ø)

[dstansby]

Is there a ruff rule we can enable to make removing these automatic in future?

[DimitriPapadopoulos]

F401 is supposed to detect such issues, and F is part of the default rulesets. However, in this specific case, ruff did not report the issue, DeepSource did.

[dstansby]

Suggested change

# flake8: noqa

If you get rid of this line, ruff should run properly on this file. I tried it locally and it seems to work.

[DimitriPapadopoulos]

This is handled in a different PR: #559 / 1c980e5.

I had to silence F401 and E402 in preview mode.