Deprecate na_action in base.map and map_arrary to not break third-party packages

Author: datapythonista

pandas/core/algorithms.py

@@ -1629,7 +1629,10 @@ def union_with_duplicates(
 
 
 def map_array(
-    arr: ArrayLike, mapper, skipna: bool = False
+    arr: ArrayLike,
+    mapper,
+    na_action: Literal["ignore"] | None | lib.NoDefault = lib.no_default,
+    skipna: bool = False,
 ) -> np.ndarray | ExtensionArray | Index:
     """
     Map values using an input mapping or function.
@@ -1638,6 +1641,11 @@ def map_array(
     ----------
     mapper : function, dict, or Series
         Mapping correspondence.
+        na_action : {None, 'ignore'}, default None
+            If 'ignore', propagate NaN values, without passing them to func.
+
+            .. deprecated:: 3.0.0
+                Use ``skipna`` instead.
     skipna : bool, default False
         If ``True``, propagate NA values, without passing them to the
         mapping correspondence.
@@ -1651,6 +1659,20 @@ def map_array(
     """
     from pandas import Index
 
+    if na_action != lib.no_default:
+        warnings.warn(
+            "The ``na_action`` parameter has been deprecated and it will be "
+            "removed in a future version of pandas. Use ``skipna`` instead.",
+            FutureWarning,
+            stacklevel=find_stack_level(),
+        )
+        if na_action == "ignore":
+            skipna = True
+        elif na_action not in (None, "ignore"):
+            raise ValueError(
+                f"na_action must either be 'ignore' or None, {na_action!r} was passed"
+            )
+
     # we can fastpath dict/Series to an efficient map
     # as we know that we are not going to have to yield
     # python types

pandas/core/arrays/base.py

@@ -2541,14 +2541,24 @@ def __array_ufunc__(self, ufunc: np.ufunc, method: str, *inputs, **kwargs):
 
         return arraylike.default_array_ufunc(self, ufunc, method, *inputs, **kwargs)
 
-    def map(self, mapper, skipna: bool = False):
+    def map(
+        self,
+        mapper,
+        na_action: Literal["ignore"] | None | lib.NoDefault = lib.no_default,
+        skipna: bool = False,
+    ):
         """
         Map values using an input mapping or function.
 
         Parameters
         ----------
         mapper : function, dict, or Series
             Mapping correspondence.
+        na_action : {None, 'ignore'}, default None
+            If 'ignore', propagate NaN values, without passing them to func.
+
+            .. deprecated:: 3.0.0
+                Use ``skipna`` instead.
         skipna : bool, default False
             If ``True``, propagate NA values, without passing them to the
             mapping correspondence. If ``True`` is not supported, a
@@ -2561,6 +2571,20 @@ def map(self, mapper, skipna: bool = False):
             If the function returns a tuple with more than one element
             a MultiIndex will be returned.
         """
+        if na_action != lib.no_default:
+            warnings.warn(
+                "The ``na_action`` parameter has been deprecated and it will be "
+                "removed in a future version of pandas. Use ``skipna`` instead.",
+                FutureWarning,
+                stacklevel=find_stack_level(),
+            )
+            if na_action == "ignore":
+                skipna = True
+            elif na_action not in (None, "ignore"):
+                raise ValueError(
+                    "na_action must either be 'ignore' or None, "
+                    f"{na_action!r} was passed"
+                )
         return map_array(self, mapper, skipna=skipna)
 
     # ------------------------------------------------------------------------