From bc47063652b4bf279db56fb90c3455c56051857b Mon Sep 17 00:00:00 2001 From: Roland Walker Date: Mon, 9 Feb 2026 04:44:43 -0500 Subject: [PATCH] reduce duplicate --checkup outputs avoid emitting a value in both the "missing" and "deprecated" sections, by checking the target of the deprecation --- changelog.md | 5 +++++ mycli/main.py | 4 ++++ 2 files changed, 9 insertions(+) diff --git a/changelog.md b/changelog.md index 6805fb51..4b7af04d 100644 --- a/changelog.md +++ b/changelog.md @@ -6,6 +6,11 @@ Features * Suggest tables/views that contain the given columns first when provided in a SELECT query. +Bug Fixes +-------- +* Reduce duplicated `--checkup` output. + + Internal -------- * Let CI ignore additional documentation files. diff --git a/mycli/main.py b/mycli/main.py index 6a8c9b9d..5c5e55bd 100755 --- a/mycli/main.py +++ b/mycli/main.py @@ -2284,6 +2284,7 @@ def do_config_checkup(mycli: MyCli) -> None: f'{indent}[main]\n{indent}default_character_set': f'{indent}[connection]\n{indent}default_character_set', f'{indent}[main]\n{indent}ssl_mode': f'{indent}[connection]\n{indent}default_ssl_mode', } + reverse_transitions = {v: k for k, v in transitions.items()} if not list(mycli.config.keys()): print('\nThe local ~/,myclirc is missing or empty.\n') @@ -2297,6 +2298,9 @@ def do_config_checkup(mycli: MyCli) -> None: did_output_missing = True continue for item_name in mycli.config[section_name]: + transition_key = f'{indent}[{section_name}]\n{indent}{item_name}' + if transition_key in reverse_transitions: + continue if item_name not in mycli.config_without_package_defaults[section_name]: if not did_output_missing: print('\n### Missing in user ~/.myclirc:\n')