fix: Make .drop(strict=...) keyword-only

`nw.DataFrame` already calls with keyword

Author: dangotbanned

narwhals/_arrow/dataframe.py

@@ -453,7 +453,7 @@ def join(
 
     join_asof = not_implemented()
 
-    def drop(self: Self, columns: Sequence[str], strict: bool) -> Self:  # noqa: FBT001
+    def drop(self: Self, columns: Sequence[str], *, strict: bool) -> Self:
         to_drop = parse_columns_to_drop(
             compliant_frame=self, columns=columns, strict=strict
         )

narwhals/_compliant/dataframe.py

@@ -66,7 +66,7 @@ def collect(
         self, backend: Implementation | None, **kwargs: Any
     ) -> CompliantDataFrame[Any]: ...
     def collect_schema(self) -> Mapping[str, DType]: ...
-    def drop(self, columns: Sequence[str], strict: bool) -> Self: ...  # noqa: FBT001
+    def drop(self, columns: Sequence[str], *, strict: bool) -> Self: ...
     def drop_nulls(self, subset: Sequence[str] | None) -> Self: ...
     def estimated_size(self, unit: SizeUnit) -> int | float: ...
     def filter(self, predicate: Any) -> Self: ...

narwhals/_pandas_like/dataframe.py

@@ -503,7 +503,7 @@ def rename(self: Self, mapping: Mapping[str, str]) -> Self:
             )
         )
 
-    def drop(self: Self, columns: Sequence[str], strict: bool) -> Self:  # noqa: FBT001
+    def drop(self: Self, columns: Sequence[str], *, strict: bool) -> Self:
         to_drop = parse_columns_to_drop(
             compliant_frame=self, columns=columns, strict=strict
         )

narwhals/_polars/dataframe.py

@@ -348,7 +348,7 @@ def with_row_index(self: Self, name: str) -> Self:
             return self._from_native_frame(self._native_frame.with_row_count(name))
         return self._from_native_frame(self._native_frame.with_row_index(name))
 
-    def drop(self: Self, columns: Sequence[str], strict: bool) -> Self:  # noqa: FBT001
+    def drop(self: Self, columns: Sequence[str], *, strict: bool) -> Self:
         to_drop = parse_columns_to_drop(
             compliant_frame=self, columns=columns, strict=strict
         )