Apply suggestions from code review

glemaitre · web-flow · commit 89339b4e74ab · 2023-03-20T10:46:05.000+01:00
diff --git a/sklearn/linear_model/_bayes.py b/sklearn/linear_model/_bayes.py
@@ -33,9 +33,10 @@ class BayesianRidge(RegressorMixin, LinearModel):
 
     Parameters
     ----------
-    max_iter : int, default=300
+    max_iter : int, default=None
         Maximum number of iterations over the complete dataset before
-        stopping independently of any early stopping criterion.
+        stopping independently of any early stopping criterion. If `None`, it
+        corresponds to `max_iter=300`.
 
         .. versionchanged:: 1.3
 
diff --git a/sklearn/linear_model/tests/test_bayes.py b/sklearn/linear_model/tests/test_bayes.py
@@ -294,9 +294,9 @@ def test_dtype_correctness(Estimator):
     np.testing.assert_allclose(coef_32, coef_64, rtol=1e-4)
 
 
+# TODO(1.5) remove
 def test_bayesian_ridge_n_iter_deprecated():
-    # check the deprecation warning of n_iter
-    # TODO(1.5) remove
+    """Check the deprecation warning of `n_iter`."""
     depr_msg = (
         "'n_iter' was renamed to 'max_iter' in version 1.3 and will be removed in 1.5"
     )
@@ -307,8 +307,9 @@ def test_bayesian_ridge_n_iter_deprecated():
         est.fit(X, y)
 
 
+# TODO(1.5) remove
 def test_bayesian_ridge_max_iter_and_n_iter_both_set():
-    # TODO(1.5) remove
+    """Check that we raise an error if both `max_iter` and `n_iter` are set."""
     """Check that a ValueError is raised when both `max_iter` and `n_iter` are set."""
     err_msg = (
         "Both `n_iter` and `max_iter` attributes were set. Attribute"
