Amend after 1st review

Author: makbigc

doc/source/whatsnew/v0.25.1.rst

@@ -18,11 +18,6 @@ Other enhancements
 
 .. _whatsnew_0251.bug_fixes:
 
-Other deprecations
-^^^^^^^^^^^^^^^^^^
-
-- The parameter ``numeric_only`` of :meth:`Categorical.min` and :meth:`Categorical.max` is deprecated and replaced with ``skipna`` (:issue:`25303`)
-
 Bug fixes
 ~~~~~~~~~
 

doc/source/whatsnew/v1.0.0.rst

@@ -54,7 +54,7 @@ Other API changes
 Deprecations
 ~~~~~~~~~~~~
 
--
+- The parameter ``numeric_only`` of :meth:`Categorical.min` and :meth:`Categorical.max` is deprecated and replaced with ``skipna`` (:issue:`25303`)
 -
 
 .. _whatsnew_1000.prior_deprecations:

pandas/tests/arrays/categorical/test_analytics.py

@@ -35,31 +35,34 @@ def test_min_max(self):
         assert _min == "d"
         assert _max == "a"
 
+    @pytest.mark.parametrize("skipna", [True, False])
+    def test_min_max_with_nan(self, skipna):
+        # GH 25303
         cat = Categorical(
             [np.nan, "b", "c", np.nan], categories=["d", "c", "b", "a"], ordered=True
         )
-        _min = cat.min(skipna=False)
-        _max = cat.max(skipna=False)
-        assert np.isnan(_min)
-        assert np.isnan(_max)
+        _min = cat.min(skipna=skipna)
+        _max = cat.max(skipna=skipna)
 
-        _min = cat.min()
-        assert _min == "c"
-        _max = cat.max()
-        assert _max == "b"
+        if skipna is False:
+            assert np.isnan(_min)
+            assert np.isnan(_max)
+        else:
+            assert _min == "c"
+            assert _max == "b"
 
         cat = Categorical(
             [np.nan, 1, 2, np.nan], categories=[5, 4, 3, 2, 1], ordered=True
         )
-        _min = cat.min(skipna=False)
-        _max = cat.max(skipna=False)
-        assert np.isnan(_min)
-        assert np.isnan(_max)
-
-        _min = cat.min()
-        assert _min == 2
-        _max = cat.max()
-        assert _max == 1
+        _min = cat.min(skipna=skipna)
+        _max = cat.max(skipna=skipna)
+
+        if skipna is False:
+            assert np.isnan(_min)
+            assert np.isnan(_max)
+        else:
+            assert _min == 2
+            assert _max == 1
 
     @pytest.mark.parametrize("method", ["min", "max"])
     def test_deprecate_numeric_only_min_max(self, method):