From 99031043286da5bd5e5d75baa4726ab7cd40b2f5 Mon Sep 17 00:00:00 2001 From: Brent Westbrook <36778786+ntBre@users.noreply.github.com> Date: Thu, 25 Sep 2025 12:14:28 -0400 Subject: [PATCH] [`pylint`] Fix missing `max-nested-blocks` in settings display (#20574) Summary -- This fixes a bug pointed out in #20560 where one of the `pylint` settings wasn't used in its `Display` implementation. Test Plan -- Existing tests with updated snapshots --- crates/ruff/tests/lint.rs | 9 +++++++++ .../show_settings__display_default_settings.snap | 1 + crates/ruff_linter/src/rules/pylint/settings.rs | 3 ++- 3 files changed, 12 insertions(+), 1 deletion(-) diff --git a/crates/ruff/tests/lint.rs b/crates/ruff/tests/lint.rs index 1d96068503..75e267e1b4 100644 --- a/crates/ruff/tests/lint.rs +++ b/crates/ruff/tests/lint.rs @@ -2445,6 +2445,7 @@ requires-python = ">= 3.11" linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -2758,6 +2759,7 @@ requires-python = ">= 3.11" linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -3070,6 +3072,7 @@ requires-python = ">= 3.11" linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -3434,6 +3437,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -3814,6 +3818,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -4142,6 +4147,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -4470,6 +4476,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -4755,6 +4762,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false @@ -5093,6 +5101,7 @@ from typing import Union;foo: Union[int, str] = 1 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 + linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false diff --git a/crates/ruff/tests/snapshots/show_settings__display_default_settings.snap b/crates/ruff/tests/snapshots/show_settings__display_default_settings.snap index cb94c10332..28a6607816 100644 --- a/crates/ruff/tests/snapshots/show_settings__display_default_settings.snap +++ b/crates/ruff/tests/snapshots/show_settings__display_default_settings.snap @@ -371,6 +371,7 @@ linter.pylint.max_branches = 12 linter.pylint.max_statements = 50 linter.pylint.max_public_methods = 20 linter.pylint.max_locals = 15 +linter.pylint.max_nested_blocks = 5 linter.pyupgrade.keep_runtime_typing = false linter.ruff.parenthesize_tuple_in_subscript = false diff --git a/crates/ruff_linter/src/rules/pylint/settings.rs b/crates/ruff_linter/src/rules/pylint/settings.rs index 383f5136c8..c3abc7e83a 100644 --- a/crates/ruff_linter/src/rules/pylint/settings.rs +++ b/crates/ruff_linter/src/rules/pylint/settings.rs @@ -96,7 +96,8 @@ impl fmt::Display for Settings { self.max_branches, self.max_statements, self.max_public_methods, - self.max_locals + self.max_locals, + self.max_nested_blocks ] } Ok(())