From patchwork Tue Jan 6 14:28:05 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 89 Return-Path: X-Original-To: u-boot-concept@u-boot.org Delivered-To: u-boot-concept@u-boot.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767709728; bh=m4g3S37Mv/c8g/1iBhLY2j6VALFMvpSPC/Nc9GnMgPE=; h=From:To:Date:CC:Subject:List-Id:List-Archive:List-Help:List-Owner: List-Post:List-Subscribe:List-Unsubscribe:From; b=NXJsEuWFIrhZIPBvmInb5cmgmpe/+mZB9pw7J0ucZCe8ufjYdiTA1giJLTPArdC+g mY2vQGdxrl1O6t7UVV5pjlKYlgfxu+Xq8i2aPyNF6+gbFF7q3Vcu2w8v2CiCOmpvrI ii8QWMXXPdiMg7KDH/0oDPxfQN5Dom72WDd9ceJjDKiUcBXCOiP5YdatAqjr7S/VpD 4SkIfhEwfe1MrUJkZ88AmXoUdMGSn007ejCexPQbQXg4vTvuIRg1xxi5M+jYhQ4n8U T3O+ErpVHqzOfQZJmyotb3Cbn24YnjYnuUFs1y3i2RPf9Bvm9UJBVzLzxhXAkEY40D 2HbTRFh/xoI7w== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 9CC516912C for ; Tue, 6 Jan 2026 07:28:48 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10024) with ESMTP id 2dEutdxpFwPl for ; Tue, 6 Jan 2026 07:28:48 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767709728; bh=m4g3S37Mv/c8g/1iBhLY2j6VALFMvpSPC/Nc9GnMgPE=; h=From:To:Date:CC:Subject:List-Id:List-Archive:List-Help:List-Owner: List-Post:List-Subscribe:List-Unsubscribe:From; b=NXJsEuWFIrhZIPBvmInb5cmgmpe/+mZB9pw7J0ucZCe8ufjYdiTA1giJLTPArdC+g mY2vQGdxrl1O6t7UVV5pjlKYlgfxu+Xq8i2aPyNF6+gbFF7q3Vcu2w8v2CiCOmpvrI ii8QWMXXPdiMg7KDH/0oDPxfQN5Dom72WDd9ceJjDKiUcBXCOiP5YdatAqjr7S/VpD 4SkIfhEwfe1MrUJkZ88AmXoUdMGSn007ejCexPQbQXg4vTvuIRg1xxi5M+jYhQ4n8U T3O+ErpVHqzOfQZJmyotb3Cbn24YnjYnuUFs1y3i2RPf9Bvm9UJBVzLzxhXAkEY40D 2HbTRFh/xoI7w== Received: from mail.u-boot.org (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 8807569122 for ; Tue, 6 Jan 2026 07:28:48 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767709725; bh=8zEJJUsWbp0gWfy1NLoiZCAbD0ySfGJtGIJxNQavgCA=; h=From:To:Cc:Subject:Date:From; b=REBlQMmfu4YQnvw7Li05yOPJxSITq7zhpIWkaFUd8zypd8pLKkTQLNup1Vv7jb8KN DVyXysMz+8shYOSQ3l56JZWAvvJN2eARv6PPPPVVkVsCmyUOm0HjDw1beT9K3XlTSC 8XL3Kv0ox1tG/a5KgePko7q5ZJxJ1wiVIkM81JB0UdOXq/8qiPt65XxqGNAbCRD5jL lVRBWUOaQDPC7zZkOLOPLfQmlyQnFms79xgtJmfmX43eIsN8xVgU4DL3HyY6+eOy37 RB9y8+JlaI5gTwHOYIsCZg5DtKg2UExIc9DKWRJrhw4cro7Lt+u95NsvT0so9rMXrP g+W2GjRrSNMXA== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id E5C0B69118; Tue, 6 Jan 2026 07:28:45 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10026) with ESMTP id TLGqCW1eerQG; Tue, 6 Jan 2026 07:28:45 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767709721; bh=Llb7cdgmVXd5Ko1VLb7zjzp6mZx/+rz4IgkfDpom8e8=; h=From:To:Cc:Subject:Date:From; b=a3yA6eBgDoQcS0SSZPMWAZ5Q4ycvc/Iprdxt5DL1A+Me9/3eF+rGk26k33fqDEDeD rpfl7f46aIKL6e/igt8iZaBLltutdBC8zMpZZjJKZWA3mBM/pxnOZGW/D+4++prdZY nRqT5q33x6FqpD3uanR8QX2AwPQgR7ZhOQWBPFnM+PeCb506fBej4VxqdA9f/kdEx/ bA4hMUcC7t3Bxi71jlr8czFQPUejckPIxwt2M1W168GsnAnJqJpfRd7CEEA5wWo0aV TxrLtPbTOFeTENIustEclEUxTh4oI5lEgB+v0W3MjIcrWlKeDoEeawp4Argjfmf6QJ vBnQ7yhJROEVg== Received: from u-boot.org (unknown [73.34.74.121]) by mail.u-boot.org (Postfix) with ESMTPSA id 7EEED69054; Tue, 6 Jan 2026 07:28:41 -0700 (MST) From: Simon Glass To: U-Boot Concept Date: Tue, 6 Jan 2026 07:28:05 -0700 Message-ID: <20260106142834.2511220-1-sjg@u-boot.org> X-Mailer: git-send-email 2.43.0 MIME-Version: 1.0 Message-ID-Hash: PC74KUGC2ZVY6ZVBNALK2FTGKPZ4O7B4 X-Message-ID-Hash: PC74KUGC2ZVY6ZVBNALK2FTGKPZ4O7B4 X-MailFrom: sjg@u-boot.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Simon Glass X-Mailman-Version: 3.3.10 Precedence: list Subject: [Concept] [PATCH 00/22] buildman: Clean up test.py for pylint compliance List-Id: Discussion and patches related to U-Boot Concept Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: Simon Glass This series addresses pylint warnings and improves code quality in buildman's test.py file. Changes include: - Fix formatting issues (indentation, long lines, f-strings) - Add missing docstrings and fix existing ones - Convert CamelCase to snake_case for consistency - Rename constants to UPPER_CASE - Split large test classes into smaller, focused classes - Refactor complex methods to reduce branch count - Disable unavoidable pylint warnings with descriptive comments Simon Glass (22): buildman: Fix indentation and semicolon in test.py buildman: Add __init__() to Options class in test.py buildman: Convert to f-strings in test.py buildman: Fix import order in test.py buildman: Convert CamelCase to snake_case in test.py buildman: Add missing docstrings in test.py buildman: Fix singleton comparisons in test.py buildman: Use tools.write_file() in test.py buildman: Use set comprehension in test.py buildman: Fix docstring in add_line_prefix() buildman: Remove unused variables in test.py buildman: Fix implicit string concatenation in test.py buildman: Mark unused arguments in test.py buildman: Disable protected-access warning in test.py buildman: Rename commit parameter to avoid shadowing in test.py buildman: Rename constants to UPPER_CASE in test.py buildman: Shorten long lines in test.py buildman: Disable pylint warnings for Options in test.py buildman: Split TestBuild into multiple classes in test.py buildman: Refactor _check_output() in test.py buildman: Split test_process_limit() into two tests in test.py buildman: Disable some final pylint warnings in test.py tools/buildman/main.py | 6 +- tools/buildman/test.py | 505 ++++++++++++++++++++++++----------------- 2 files changed, 303 insertions(+), 208 deletions(-)