Did the necessary docstring cleanup as discussed in #590

earthaccess/auth.py

@@ -47,9 +47,8 @@ def __init__(
         if username and password:
             self.auth = (username, password)
 
-    # Overrides from the library to keep headers when redirected to or from
-    # the NASA auth host.
     def rebuild_auth(self, prepared_request: Any, response: Any) -> None:
+        """Overrides from the library to keep headers when redirected to or from the NASA auth host."""
         headers = prepared_request.headers
         url = prepared_request.url
 
@@ -93,14 +92,16 @@ def login(
                 * **"environment"**:
                     Retrieve a username and password from $EARTHDATA_USERNAME and $EARTHDATA_PASSWORD.
             persist: Will persist credentials in a `.netrc` file.
-            system (Env): the EDL endpoint to log in to Earthdata, defaults to PROD
 
         Returns:
             An instance of Auth.
         """
         if system is not None:
             self._set_earthdata_system(system)
 
+        if system is not None:
+            self._set_earthdata_system(system)
+
         if self.authenticated and (system == self.system):
             logger.debug("We are already authenticated with NASA EDL")
             return self
@@ -132,6 +133,7 @@ def _set_earthdata_system(self, system: System) -> None:
 
     def refresh_tokens(self) -> bool:
         """Refresh CMR tokens.
+
         Tokens are used to do authenticated queries on CMR for restricted and early access datasets.
         This method renews the tokens to make sure we can query the collections allowed to our EDL user.
         """
@@ -188,6 +190,7 @@ def get_s3_credentials(
         endpoint: Optional[str] = None,
     ) -> Dict[str, str]:
         """Gets AWS S3 credentials for a given NASA cloud provider.
+
         The easier way is to use the DAAC short name; provider is optional if we know it.
 
         Parameters:

earthaccess/daac.py

@@ -133,6 +133,7 @@
 def find_provider(
     daac_short_name: Optional[str] = None, cloud_hosted: Optional[bool] = None
 ) -> Union[str, None]:
+    """Placeholder."""
     for daac in DAACS:
         if daac_short_name == daac["short-name"]:
             if cloud_hosted:
@@ -148,6 +149,7 @@ def find_provider(
 
 
 def find_provider_by_shortname(short_name: str, cloud_hosted: bool) -> Union[str, None]:
+    """Placeholder."""
     base_url = "https://cmr.earthdata.nasa.gov/search/collections.umm_json?"
     providers = requests.get(
         f"{base_url}&cloud_hosted={cloud_hosted}&short_name={short_name}"

earthaccess/kerchunk.py

@@ -29,6 +29,7 @@ def consolidate_metadata(
     outfile: str | None = None,
     storage_options: dict | None = None,
 ) -> str | dict:
+    """Placeholder."""
     try:
         import dask
 

earthaccess/search.py

@@ -114,8 +114,9 @@ def __init__(self, auth: Optional[Auth] = None, *args: Any, **kwargs: Any) -> No
 
     @override
     def hits(self) -> int:
-        """Returns the number of hits the current query will return. This is done by
-        making a lightweight query to CMR and inspecting the returned headers.
+        """Returns the number of hits the current query will return.
+
+        This is done by making a lightweight query to CMR and inspecting the returned headers.
         Restricted datasets will always return zero results even if there are results.
 
         Returns:
@@ -469,6 +470,7 @@ def __init__(self, auth: Optional[Auth] = None, *args: Any, **kwargs: Any) -> No
     @override
     def hits(self) -> int:
         """Returns the number of hits the current query will return.
+
         This is done by making a lightweight query to CMR and inspecting the returned
         headers.
 
@@ -561,6 +563,7 @@ def parameters(self, **kwargs: Any) -> Self:
     @override
     def provider(self, provider: str) -> Self:
         """Only match collections from a given provider.
+
         A NASA datacenter or DAAC can have one or more providers.
         For example, PODAAC is a data center or DAAC,
         PODAAC is the default provider for on-prem data, and POCLOUD is

earthaccess/store.py

@@ -62,6 +62,7 @@ def make_instance(
     cls: Any, granule: DataGranule, auth: Auth, data: Any
 ) -> EarthAccessFile:
     # Attempt to re-authenticate
+    """Placeholder."""
     if not earthaccess.__auth__.authenticated:
         earthaccess.__auth__ = auth
         earthaccess.login()
@@ -165,6 +166,7 @@ def set_requests_session(
         self, url: str, method: str = "get", bearer_token: bool = False
     ) -> None:
         """Sets up a `requests` session with bearer tokens that are used by CMR.
+
         Mainly used to get the authentication cookies from different DAACs and URS.
         This HTTPS session can be used to download granules if we want to use a direct,
         lower level API.
@@ -265,6 +267,7 @@ def get_s3fs_session(
     @lru_cache
     def get_fsspec_session(self) -> fsspec.AbstractFileSystem:
         """Returns a fsspec HTTPS session with bearer tokens that are used by CMR.
+
         This HTTPS session can be used to download granules if we want to use a direct,
         lower level API.
 
@@ -283,6 +286,7 @@ def get_fsspec_session(self) -> fsspec.AbstractFileSystem:
 
     def get_requests_session(self, bearer_token: bool = True) -> requests.Session:
         """Returns a requests HTTPS session with bearer tokens that are used by CMR.
+
         This HTTPS session can be used to download granules if we want to use a direct,
         lower level API.