diff mbox series

[Concept,01/16] pickman: Drop unnecessary f-string prefixes in do_rewind()

Message ID 20260222154303.2851319-2-sjg@u-boot.org
State New
Headers
Series pickman: Support monitoring and fixing pipeline failures |

Commit Message

Simon Glass Feb. 22, 2026, 3:42 p.m. UTC 
  From: Simon Glass <simon.glass@canonical.com>

Three tout.info() calls in do_rewind() use f-strings that contain no
interpolated variables, triggering pylint W1309. Remove the f prefix
from these plain strings.

Co-developed-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/pickman/control.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/tools/pickman/control.py b/tools/pickman/control.py
index 8f7568e47bf..40bcaaef120 100644
--- a/tools/pickman/control.py
+++ b/tools/pickman/control.py
@@ -1952,18 +1952,18 @@  def do_rewind(args, dbs):
     tout.info(f"{prefix}Rewind '{source}': "
               f'{current_short} -> {target_chash}')
     tout.info(f'  Target: {target_chash} {target_subject}')
-    tout.info(f'  Merges being rewound:')
+    tout.info('  Merges being rewound:')
     for i in range(target_idx):
         tout.info(f'    {merges[i][1]} {merges[i][2]}')
     tout.info(f'  Commits to delete from database: {len(db_commits)}')
 
     if matched_mrs:
-        tout.info(f'  MRs to delete on GitLab:')
+        tout.info('  MRs to delete on GitLab:')
         for merge_req in matched_mrs:
             tout.info(f'    !{merge_req.iid}: {merge_req.title}')
             tout.info(f'      {merge_req.web_url}')
     elif mr_branches:
-        tout.info(f'  Branches to check for MRs:')
+        tout.info('  Branches to check for MRs:')
         for branch in mr_branches:
             tout.info(f'    {branch}')