refactor(api)!: align signatures of truncate_table method; name is positional-only; the rest are keyword-only

Author: cpcloud

ibis/backends/clickhouse/__init__.py

@@ -575,7 +575,7 @@ def drop_database(self, name: str, /, *, force: bool = False) -> None:
         with self._safe_raw_sql(src):
             pass
 
-    def truncate_table(self, name: str, database: str | None = None) -> None:
+    def truncate_table(self, name: str, /, *, database: str | None = None) -> None:
         ident = sg.table(name, db=database).sql(self.name)
         with self._safe_raw_sql(f"TRUNCATE TABLE {ident}"):
             pass

ibis/backends/datafusion/__init__.py

@@ -666,11 +666,7 @@ def create_table(
 
         return self.table(name, database=database)
 
-    def truncate_table(
-        self,
-        name: str,
-        database: str | None = None,
-    ):
+    def truncate_table(self, name: str, /, *, database: str | None = None):
         """Delete all rows from a table.
 
         Parameters
@@ -679,7 +675,6 @@ def truncate_table(
             Table name
         database
             Database name
-
         """
         # datafusion doesn't support `TRUNCATE TABLE` so we use `DELETE FROM`
         #

ibis/backends/impala/__init__.py

@@ -836,7 +836,7 @@ def drop_table(
         statement = ddl.DropTable(name, database=database, must_exist=not force)
         self._safe_exec_sql(statement)
 
-    def truncate_table(self, name: str, database: str | None = None) -> None:
+    def truncate_table(self, name: str, /, *, database: str | None = None) -> None:
         """Delete all rows from an existing table.
 
         Parameters
@@ -845,7 +845,6 @@ def truncate_table(self, name: str, database: str | None = None) -> None:
             Table name
         database
             Database name
-
         """
         statement = ddl.TruncateTable(name, database=database)
         self._safe_exec_sql(statement)

ibis/backends/sql/__init__.py

@@ -454,11 +454,7 @@ def _build_insert_template(
             ),
         ).sql(self.dialect)
 
-    def truncate_table(
-        self,
-        name: str,
-        database: str | None = None,
-    ) -> None:
+    def truncate_table(self, name: str, /, *, database: str | None = None) -> None:
         """Delete all rows from a table.
 
         ::: {.callout-note}