Change which_browser to browser_which

Author: ayjayt

choreographer/__init__.py

@@ -1,7 +1,13 @@
 from .browser import Browser
+from .browser import browser_which
 from .browser import get_browser_path
-from .browser import which_browser
 from .cli_utils import get_browser
 from .cli_utils import get_browser_sync
 
-__all__ = [Browser, get_browser, get_browser_sync, which_browser, get_browser_path]
+__all__ = [
+    Browser,
+    get_browser,
+    get_browser_sync,
+    browser_which,
+    get_browser_path,
+]

choreographer/cli_utils.py

@@ -149,7 +149,7 @@ async def get_browser(
 
 
 def diagnose():
-    from choreographer import Browser, which_browser
+    from choreographer import Browser, browser_which
 
     parser = argparse.ArgumentParser(description="tool to help debug problems")
     parser.add_argument("--no-run", dest="run", action="store_false")
@@ -167,7 +167,7 @@ def diagnose():
     print(platform.version())
     print(platform.uname())
     print("BROWSER:".center(50, "*"))
-    print(which_browser(debug=True))
+    print(browser_which(debug=True))
     print("VERSION INFO:".center(50, "*"))
     try:
         print("PIP:".center(25, "*"))

choreographer/system.py

@@ -70,7 +70,7 @@ def _is_exe(path):
         return res
 
 
-def which_browser(executable_name=chrome, debug=False, skip_local=False):
+def browser_which(executable_name=chrome, debug=False, skip_local=False):
     if debug:
         print(f"Checking {default_exe_name}", file=sys.stderr)
     if not skip_local and os.path.exists(default_exe_name):