Enable F401 (pyflakes unused imports) and TRY003 (flake8-tryceratops) rulesets and fix violations (#1361)

Co-authored-by: Matt Fisher <3608264+mfisher87@users.noreply.github.com>

Author: Sherwin-14

earthaccess/__init__.py

@@ -1,6 +1,5 @@
 import logging
 import threading
-from typing import Optional
 
 from .api import (
     auth_environ,

earthaccess/virtual/_types.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 from collections.abc import Callable
-from typing import Any, Literal, Union
+from typing import Any, Literal
 
 # A VirtualiZarr parser: either a canonical class-name string or a pre-built
 # parser instance.  The strings are the exact class names exported from

pyproject.toml

@@ -210,7 +210,6 @@ ignore = [
   "D401",
   "D417",
   "E501",
-  "F401",
   "FIX002",
   "PGH003",
   "PT006",
@@ -228,16 +227,15 @@ ignore = [
   "S607",
   "TD002",
   "TD003",
-  "TD004",
-  "TRY003",
+  "TD004"
 ]
 
 [tool.ruff.lint.isort]
 combine-as-imports = true
 
 [tool.ruff.lint.per-file-ignores]
 # Allow `print` and `pprint` in notebooks
-"*.ipynb" = ["T201", "T203", "N803", "RET504", "ERA001", "PLR2004", "PD002", "PD010", "PD011", "EM", "ARG001", "BLE001", "FBT"]
+"*.ipynb" = ["T201", "T203", "N803", "RET504", "ERA001", "PLR2004", "PD002", "PD010", "PD011", "EM", "ARG001", "BLE001", "FBT","TRY003"]
 "tests/**/*.py" = ["PLR2004", "FBT", "INP001", "SLF001"]
 "scripts/**/*.py" = ["INP001"]
 

stubs/cmr/queries.pyi

@@ -1,7 +1,6 @@
-import sys
 from collections.abc import MutableMapping, Sequence
 from datetime import date, datetime
-from typing import Any, Self, SupportsFloat, TypeAlias
+from typing import Any, Self, SupportsFloat
 
 CMR_OPS: str
 CMR_UAT: str

tests/integration/test_cloud_download.py

@@ -1,6 +1,5 @@
 import logging
 import operator as op
-import os
 import shutil
 from collections.abc import Callable
 from pathlib import Path