refactor(api)!: offset argument in Table.limit is now required-keyword

Author: cpcloud

ibis/expr/decompile.py

@@ -234,7 +234,7 @@ def union(op, left, right, distinct):
 @translate.register(ops.Limit)
 def limit(op, parent, n, offset):
     if offset:
-        return f"{parent}.limit({n}, {offset})"
+        return f"{parent}.limit({n}, offset={offset})"
     else:
         return f"{parent}.limit({n})"
 

ibis/expr/types/relations.py

@@ -1298,7 +1298,7 @@ def sample(
                 self, fraction=fraction, method=method, seed=seed
             ).to_expr()
 
-    def limit(self, n: int | None, offset: int = 0) -> Table:
+    def limit(self, n: int | None, *, offset: int = 0) -> Table:
         """Select `n` rows from `self` starting at `offset`.
 
         ::: {.callout-note}