Switch to Ruff

[jhkennedy]

this resolves #341 by dropping autoflake, black, flake8, and isort in favor of ruff

There were some minor issues caught by ruff that I fixed as well as some differences in formatting, which IMO are improvements (this is subjective).

Notably, flake8 had a line length set at 120 while isort had it set at 88. With ruff, those are both set at 120 and there are some white-space changes, especially around unnecessarily breaking things that are shorter than 120.

I've noted any changes that are more than whitespace in the PR.

[github-actions]

👈 Launch a binder notebook on this branch for commit 7057173

I will automatically update this comment whenever this PR is modified

👈 Launch a binder notebook on this branch for commit ccd14ac

👈 Launch a binder notebook on this branch for commit 6fa4aee

👈 Launch a binder notebook on this branch for commit 9845607

👈 Launch a binder notebook on this branch for commit e5899f5

👈 Launch a binder notebook on this branch for commit 7b2d5d1

👈 Launch a binder notebook on this branch for commit 7ed8385

[jhkennedy]

Looks like this workflow doesn't run here b/c it was added on a fork -- you can see it run in my fork here:
https://github.com/jhkennedy/earthdata/actions/workflows/static-analysis.yml

I added this in addition to it in the pre-commit because it will annotate the PR, which you can see in this PR:
https://github.com/ASFHyP3/hyp3-isce2/pull/156/files

[jhkennedy]

$ ruff check earthaccess tests
earthaccess/store.py:55:12: E721 Do not compare types, use `isinstance()`

[MattF-NSIDC]

I would prefer to remove the repeated (and I think unneeded) earthaccess tests arguments to ruff. If you prefer not to do that in this PR, we can always get it in another one.

[jhkennedy]

@mfisher87 switched earthaccess tests -> .. I think this is g2g now?

Stale

[MattF-NSIDC]

Thanks! Looks amazing :) 💯