feat(api): make more connect() params kwarg-only or positional-only

Author: NickCrews

ibis/backends/bigquery/__init__.py

@@ -339,6 +339,7 @@ def do_connect(
         self,
         project_id: str | None = None,
         dataset_id: str = "",
+        *,
         credentials: google.auth.credentials.Credentials | None = None,
         application_name: str | None = None,
         auth_local_webserver: bool = True,

ibis/backends/clickhouse/__init__.py

@@ -107,6 +107,7 @@ def _convert_kwargs(self, kwargs):
 
     def do_connect(
         self,
+        *,
         host: str = "localhost",
         port: int | None = None,
         database: str = "default",

ibis/backends/datafusion/__init__.py

@@ -87,7 +87,7 @@ def version(self):
         return importlib.metadata.version("datafusion")
 
     def do_connect(
-        self, config: Mapping[str, str | Path] | SessionContext | None = None
+        self, config: Mapping[str, str | Path] | SessionContext | None = None, /
     ) -> None:
         """Create a DataFusion `Backend` for use with Ibis.
 

ibis/backends/duckdb/__init__.py

@@ -365,6 +365,8 @@ def version(self) -> str:
     def do_connect(
         self,
         database: str | Path = ":memory:",
+        /,
+        *,
         read_only: bool = False,
         extensions: Sequence[str] | None = None,
         **config: Any,

ibis/backends/exasol/__init__.py

@@ -57,6 +57,7 @@ def version(self) -> str:
 
     def do_connect(
         self,
+        *,
         user: str,
         password: str,
         host: str = "localhost",

ibis/backends/impala/__init__.py

@@ -90,6 +90,7 @@ def _from_url(self, url: ParseResult, **kwargs: Any) -> Backend:
 
     def do_connect(
         self,
+        *,
         host: str = "localhost",
         port: int = 21050,
         database: str = "default",

ibis/backends/mssql/__init__.py

@@ -88,6 +88,7 @@ def version(self) -> str:
 
     def do_connect(
         self,
+        *,
         host: str = "localhost",
         user: str | None = None,
         password: str | None = None,

ibis/backends/mysql/__init__.py

@@ -91,6 +91,7 @@ def version(self):
 
     def do_connect(
         self,
+        *,
         host: str = "localhost",
         user: str | None = None,
         password: str | None = None,

ibis/backends/polars/__init__.py

@@ -38,7 +38,7 @@ def __init__(self, *args, **kwargs):
         self._context = pl.SQLContext()
 
     def do_connect(
-        self, tables: Mapping[str, pl.LazyFrame | pl.DataFrame] | None = None
+        self, tables: Mapping[str, pl.LazyFrame | pl.DataFrame] | None = None, /
     ) -> None:
         """Construct a client from a dictionary of polars `LazyFrame`s and/or `DataFrame`s.
 

ibis/backends/postgres/__init__.py

@@ -201,6 +201,7 @@ def version(self):
 
     def do_connect(
         self,
+        *,
         host: str | None = None,
         user: str | None = None,
         password: str | None = None,

ibis/backends/pyspark/__init__.py

@@ -137,6 +137,7 @@ def __init__(self, *args, **kwargs):
     def do_connect(
         self,
         session: SparkSession | None = None,
+        *,
         mode: ConnectionMode = "batch",
         **kwargs,
     ) -> None:

ibis/backends/risingwave/__init__.py

@@ -431,6 +431,7 @@ def _safe_raw_sql(self, *args, **kwargs):
 
     def do_connect(
         self,
+        *,
         host: str | None = None,
         user: str | None = None,
         password: str | None = None,

ibis/backends/snowflake/__init__.py

@@ -239,7 +239,7 @@ def _make_udf(self, name: str, defn) -> str:
 AS
 $$ {defn["source"]} $$"""
 
-    def do_connect(self, create_object_udfs: bool = True, **kwargs: Any):
+    def do_connect(self, *, create_object_udfs: bool = True, **kwargs: Any):
         """Connect to Snowflake.
 
         Parameters

ibis/backends/sqlite/__init__.py

@@ -59,6 +59,8 @@ def version(self) -> str:
     def do_connect(
         self,
         database: str | Path | None = None,
+        /,
+        *,
         type_map: dict[str, str | dt.DataType] | None = None,
     ) -> None:
         """Create an Ibis client connected to a SQLite database.

ibis/backends/trino/__init__.py

@@ -243,6 +243,7 @@ def list_tables(
 
     def do_connect(
         self,
+        *,
         user: str = "user",
         password: str | None = None,
         host: str = "localhost",