Refactor function signatures in env_cleaner.py to remove unused parameters and improve docstring formatting

Author: sathiya-nhs

env_cleaner.py

@@ -19,13 +19,13 @@
   --min-age=<min_age>            (NOT IMPLEMENTED) Minimum age in seconds
   --respect-prs                  Don't undeploy if there's a PR open for it
 """
+
 import re
 import requests
 from typing import Optional, Set
 from docopt import docopt
 from apigee_client import ApigeeClient
 
-
 SPEC_MATCHER = re.compile("(personal-demographics|identity-service|hello-world)-.+")
 PROXY_MATCHER = re.compile(
     "(personal-demographics|identity-service|hello-world)-internal-dev-.+"
@@ -49,7 +49,7 @@ def canonicalize(name: str) -> str:
 
 class GithubClient:
     def get_open_prs(self, env: str) -> Set[str]:
-        """ Returns names of open pull requests """
+        """Returns names of open pull requests"""
         canonical_prs = set()
         for repo_name in REPO_NAMES.keys():
             prs = requests.get(
@@ -65,7 +65,7 @@ def get_open_prs(self, env: str) -> Set[str]:
         return canonical_prs
 
 
-def clean_specs(client: ApigeeClient, env: str, dry_run: bool = False):
+def clean_specs(client: ApigeeClient, dry_run: bool = False):
     specs = client.list_specs()
     spec_names = [spec["name"] for spec in specs["contents"]]
 
@@ -85,7 +85,6 @@ def clean_proxies(
     env: str,
     dry_run: bool = False,
     sandboxes_only: bool = False,
-    min_age: Optional[int] = None,
     undeploy_only: bool = False,
     respect_prs: bool = False,
 ):
@@ -135,7 +134,7 @@ def clean_proxies(
                 client.delete_proxy(proxy)
 
 
-def clean_products(client: ApigeeClient, env: str, dry_run: bool = False):
+def clean_products(client: ApigeeClient, dry_run: bool = False):
     products = client.list_products()
     pr_products = [product for product in products if PRODUCT_MATCHER.match(product)]