corteva / rioxarray

geospatial xarray extension powered by rasterio
https://corteva.github.io/rioxarray
Other
504 stars 80 forks source link

write somewhere in the documentation that extentions are not respecting F401 #734

Open 12rambau opened 5 months ago

12rambau commented 5 months ago

When running linting process using Ruff or flake8, F401 will remove any extension lib from the file as its never called. I checked your own codebase and you are specifying:

import rioxarray # noqa: F401

Would it make sense to notify potential users in the documentation ?

I was using auto-fix from ruff in my project and it took me a while to understand that the pre-commit was removing rioxarray from the import sections all the times :smile: