options,hooks(refactor): Replace g param deprecation warning with error

why: Enforce migration from deprecated 'g' parameter to 'global_'.

what:
- options.py: set_option(), _show_options_raw(), _show_option_raw(),
  show_option() now raise DeprecatedError when g parameter is used
- hooks.py: set_hook() now raises DeprecatedError when g parameter is used

Author: tony

src/libtmux/hooks.py

@@ -47,6 +47,8 @@
 )
 from libtmux.options import handle_option_error
 
+from . import exc
+
 if t.TYPE_CHECKING:
     from typing_extensions import Self
 
@@ -139,12 +141,11 @@ def set_hook(
             scope = self.default_hook_scope
 
         if g:
-            warnings.warn(
-                "g argument is deprecated in favor of global_",
-                category=DeprecationWarning,
-                stacklevel=2,
+            raise exc.DeprecatedError(
+                deprecated="g parameter",
+                replacement="global_ parameter",
+                version="0.50.0",
             )
-            global_ = g
 
         flags: list[str] = []
 

src/libtmux/options.py

@@ -72,7 +72,6 @@
 import re
 import shlex
 import typing as t
-import warnings
 
 from libtmux._internal.sparse_array import SparseArray
 from libtmux.common import CmdMixin
@@ -669,12 +668,11 @@ def set_option(
             flags.append("-a")
 
         if g is not None:
-            warnings.warn(
-                "g argument is deprecated in favor of global_",
-                category=DeprecationWarning,
-                stacklevel=2,
+            raise exc.DeprecatedError(
+                deprecated="g parameter",
+                replacement="global_ parameter",
+                version="0.50.0",
             )
-            global_ = g
 
         if global_ is not None and global_:
             assert isinstance(global_, bool)
@@ -829,13 +827,13 @@ def _show_options_raw(
         flags: tuple[str, ...] = ()
 
         if g:
-            warnings.warn(
-                "g argument is deprecated in favor of global_",
-                category=DeprecationWarning,
-                stacklevel=2,
+            raise exc.DeprecatedError(
+                deprecated="g parameter",
+                replacement="global_ parameter",
+                version="0.50.0",
             )
-            flags += ("-g",)
-        elif global_:
+
+        if global_:
             flags += ("-g",)
 
         if scope is not None and not isinstance(scope, _DefaultOptionScope):
@@ -1058,13 +1056,13 @@ def _show_option_raw(
         flags: tuple[str | int, ...] = ()
 
         if g:
-            warnings.warn(
-                "g argument is deprecated in favor of global_",
-                category=DeprecationWarning,
-                stacklevel=2,
+            raise exc.DeprecatedError(
+                deprecated="g parameter",
+                replacement="global_ parameter",
+                version="0.50.0",
             )
-            flags += ("-g",)
-        elif global_:
+
+        if global_:
             flags += ("-g",)
 
         if scope is not None and not isinstance(scope, _DefaultOptionScope):
@@ -1237,12 +1235,11 @@ def show_option(
         False
         """
         if g:
-            warnings.warn(
-                "g argument is deprecated in favor of global_",
-                category=DeprecationWarning,
-                stacklevel=2,
+            raise exc.DeprecatedError(
+                deprecated="g parameter",
+                replacement="global_ parameter",
+                version="0.50.0",
             )
-            global_ = g
 
         return self._show_option(
             option=option,