From 3723ee7b751106956b155b1b5f8f38c25e34755e Mon Sep 17 00:00:00 2001 From: Pierre Sassoulas Date: Mon, 8 Nov 2021 10:27:18 +0100 Subject: Simplify toml handling by removing else after try --- pylint/config/option_manager_mixin.py | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/pylint/config/option_manager_mixin.py b/pylint/config/option_manager_mixin.py index 34e0b3042..6720bc40e 100644 --- a/pylint/config/option_manager_mixin.py +++ b/pylint/config/option_manager_mixin.py @@ -302,19 +302,18 @@ class OptionsManagerMixIn: try: sections_values = content["tool"]["pylint"] except KeyError: - pass - else: - for section, values in sections_values.items(): - # TOML has rich types, convert values to - # strings as ConfigParser expects. - for option, value in values.items(): - if isinstance(value, bool): - values[option] = "yes" if value else "no" - elif isinstance(value, (int, float)): - values[option] = str(value) - elif isinstance(value, list): - values[option] = ",".join(value) - parser._sections[section.upper()] = values # type: ignore + return + for section, values in sections_values.items(): + # TOML has rich types, convert values to + # strings as ConfigParser expects. + for option, value in values.items(): + if isinstance(value, bool): + values[option] = "yes" if value else "no" + elif isinstance(value, (int, float)): + values[option] = str(value) + elif isinstance(value, list): + values[option] = ",".join(value) + parser._sections[section.upper()] = values # type: ignore def load_config_file(self): """Dispatch values previously read from a configuration file to each -- cgit v1.2.1