CLN: Refactor groupby._make_wrapper

pandas/core/apply.py

@@ -550,10 +550,12 @@ def apply_str(self) -> DataFrame | Series:
         func = getattr(obj, f, None)
         if callable(func):
             sig = inspect.getfullargspec(func)
-            if "axis" in sig.args:
-                self.kwargs["axis"] = self.axis
-            elif self.axis != 0:
+            if self.axis != 0 and (
+                "axis" not in sig.args or f in ("corrwith", "mad", "skew")
+            ):
                 raise ValueError(f"Operation {f} does not support axis=1")
+            elif "axis" in sig.args:
+                self.kwargs["axis"] = self.axis
         return self._try_aggregate_string_function(obj, f, *self.args, **self.kwargs)
 
     def apply_multiple(self) -> DataFrame | Series:

pandas/core/generic.py

@@ -11657,10 +11657,8 @@ def all(self, axis=0, bool_only=None, skipna=True, level=None, **kwargs):
 
         setattr(cls, "all", all)
 
-        # error: Argument 1 to "doc" has incompatible type "Optional[str]"; expected
-        # "Union[str, Callable[..., Any]]"
         @doc(
-            NDFrame.mad.__doc__,  # type: ignore[arg-type]
+            NDFrame.mad.__doc__,
             desc="Return the mean absolute deviation of the values "
             "over the requested axis.",
             name1=name1,

pandas/core/groupby/base.py

@@ -1,7 +1,5 @@
 """
-Provide basic components for groupby. These definitions
-hold the allowlist of methods that are exposed on the
-SeriesGroupBy and the DataFrameGroupBy objects.
+Provide basic components for groupby.
 """
 from __future__ import annotations
 
@@ -22,36 +20,6 @@ class OutputKey:
 # forwarding methods from NDFrames
 plotting_methods = frozenset(["plot", "hist"])
 
-common_apply_allowlist = (
-    frozenset(
-        [
-            "quantile",
-            "fillna",
-            "mad",
-            "take",
-            "idxmax",
-            "idxmin",
-            "tshift",
-            "skew",
-            "corr",
-            "cov",
-            "diff",
-        ]
-    )
-    | plotting_methods
-)
-
-series_apply_allowlist: frozenset[str] = (
-    common_apply_allowlist
-    | frozenset(
-        {"nlargest", "nsmallest", "is_monotonic_increasing", "is_monotonic_decreasing"}
-    )
-) | frozenset(["dtype", "unique"])
-
-dataframe_apply_allowlist: frozenset[str] = common_apply_allowlist | frozenset(
-    ["dtypes", "corrwith"]
-)
-
 # cythonized transformations or canned "agg+broadcast", which do not
 # require postprocessing of the result by transform.
 cythonized_kernels = frozenset(["cumprod", "cumsum", "shift", "cummin", "cummax"])