From patchwork Sat Apr 4 21:28:59 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 2138 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=1775338358; bh=38zaXjuHr/z6qG/ZR5GM6BCDfSjVG+j+D4lqvU6PD3k=; h=From:To:Date:In-Reply-To:References:CC:Subject:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=Xzjt/kmhY3I0hF5DVsH1nGA8i/pmJLWrGZ2rTqjLM3nD7sLsJDCTOIZ56YJdxVX2h OM3CRtk6XQB/ms04EfnQhnsc0k6YBKJOQNNk0nKaJE3Xyr5vATKIPdtTN/ClxBa4D5 ri2AmpvQC06X6p1m4W4+lTT2fUABtNSgX2NeH6/7KPPIJQDuEqtDY5cSpKmz7ManPq 6cF0/orFJD7tmvy5JgdOgoZAjga7w20zZ4SAUEQcmBCTjV2Cqb9ELTRG63UaDK4EEu l1lDcL8xyEjCLi1Q4CLLhyhEwrzpPW5tS9l4iStfHUMxJJ2ejiiN+z8q5t0/HA+jjv Gh/ajaxNaCOvA== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 032BA68F7F for ; Sat, 4 Apr 2026 15:32:38 -0600 (MDT) 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 olPx5PQ2oroh for ; Sat, 4 Apr 2026 15:32:37 -0600 (MDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1775338357; bh=38zaXjuHr/z6qG/ZR5GM6BCDfSjVG+j+D4lqvU6PD3k=; h=From:To:Date:In-Reply-To:References:CC:Subject:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=RXKH+jZ6oauposm2AeUZEBkyvnnyN7XKkfbMvWwdW5VQ8C/mM365gX/0h6NziSxcZ BInKSLh4UZhtBlXSz6cFCVghCA4EV+05dP98oRT0aXgSsJAP0h4aE2wXKnlEsyuNVY YORWQi9nJwgaE3fCZx71hkwYlnbGj4aHMRjuJNHVTCZzxutX5BiXxd8ruLI4PGg/cJ GCGpn9Np/IYIb4LY3jUoGjM+hVg2mftKIOzbx3qG2AaP/FScsDbsmqNUtcE5dmQmvK hRco+/jRTBgf10kgGLOL7u2IqGiBPFI+XONVnbkUjPgjD2l2VExlTqC2SwU0uFn/CZ nH3FSBfvzGCcg== Received: from mail.u-boot.org (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id E67C06833B for ; Sat, 4 Apr 2026 15:32:37 -0600 (MDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1775338356; bh=i6gHcorQFWk3ePANhzenV+q6IMcq0CdljNU95AKnqhA=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=tZPKUGqWJg+uWXJ9ND2yXBBldmGH5KgvKBVXCLfZiYC3CZyoIi4J8QWxZVahG9FNC fl4QHs7PpGazhOG2mAT58sodomxkAQsnizeSBYINdDQiKUtTFKRyrevVKa+srKvgzB MVb/Fl9LzJGlP+ubd8nhY2hOqZYlh0njjJTj8eUQdCTURaTxO2BYfChG7rHX+Zynyw AIIlOr0inQ7PLoNNpfSOZk6JYNvoG07K3X8dhzE3vNPKMLaQrf1ctDpQz3ZMOoBurm ki7m4W8NIUIB7t4kf58y++uPsBvs7Zs+9WhSYA/+ORi3oh5Xwlks82Y9jGBZ7H6lmC BeNqx5HsDhdtQ== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 1D1F75FC8F; Sat, 4 Apr 2026 15:32:36 -0600 (MDT) 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 JpBMchREtjde; Sat, 4 Apr 2026 15:32:36 -0600 (MDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1775338351; bh=K9FFsB0fOQDeNaUASpr63jKh3WpsshDXLwXFR3/S95I=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=wnk7GDlaVYpt3nuZWRkKj42yloKopeL8brZHl3yu15eWX4dnZ3dda1tgHcdZDT/Xi nnp+SYEMWpd0S6ZRYnaVX67RakoA5jEJoNe4kNSIZNhfNDw9pX1zveTMq+JBBNgEIX TfqRKWwXQUaGzVEdpEqtx2K1wJ/vfsZIXn2n1c4dg5JsgatUQrcTIcnTJGCsvM5VXT Mi/DHvX7smrbPLQVuUj1mhmw/VKNrduLxD6aZBidrMlFKFKlBSFAj2WHyMqH20WJHg +gruUkXxlVepzcdjRKjv+KShDfTIaefiTIOHOFIKDWvwSvZ14LTVuQBxmeI0pq1F3s 9Z+jUqQPWKZWA== Received: from u-boot.org (unknown [73.34.74.121]) by mail.u-boot.org (Postfix) with ESMTPSA id B564C5E7B4; Sat, 4 Apr 2026 15:32:31 -0600 (MDT) From: Simon Glass To: U-Boot Concept Date: Sat, 4 Apr 2026 15:28:59 -0600 Message-ID: <20260404213020.372253-24-sjg@u-boot.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260404213020.372253-1-sjg@u-boot.org> References: <20260404213020.372253-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: S3U4P627XWR2I3ZV44TEMK7C44VK7LTS X-Message-ID-Hash: S3U4P627XWR2I3ZV44TEMK7C44VK7LTS 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 23/37] patman: Fix remaining pylint warnings in control.py 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 Add missing single_thread parameter to patchwork_status() docstring, wrap a long error message to fit 80 columns, and suppress R0917 (too-many-positional-arguments) introduced in pylint 3.3.4. Signed-off-by: Simon Glass --- tools/patman/control.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/tools/patman/control.py b/tools/patman/control.py index 1c1b998ddd6..d82b80ce807 100644 --- a/tools/patman/control.py +++ b/tools/patman/control.py @@ -48,7 +48,7 @@ def do_send(args): send.send(args) -# pylint: disable=R0913 +# pylint: disable=R0913,R0917 def patchwork_status(branch, count, start, end, dest_branch, force, show_comments, url, single_thread=False): """Check the status of patches in patchwork @@ -69,8 +69,11 @@ def patchwork_status(branch, count, start, end, dest_branch, force, force (bool): With dest_branch, force overwriting an existing branch show_comments (bool): True to display snippets from the comments provided by reviewers - url (str): URL of patchwork server, e.g. 'https://patchwork.ozlabs.org'. - This is ignored if the series provides a Series-patchwork-url tag. + url (str): URL of patchwork server, e.g. + 'https://patchwork.ozlabs.org'. Ignored if the series + provides a Series-patchwork-url tag. + single_thread (bool): True to use a single thread for + patchwork access Raises: ValueError: if the branch has no Series-link value @@ -94,7 +97,7 @@ def patchwork_status(branch, count, start, end, dest_branch, force, raise ValueError('Please fix warnings before running status') links = series.get('links') if not links: - raise ValueError("Branch has no Series-links value") + raise ValueError('Branch has no Series-links value') status.find_link_and_show_status( series, branch, url, dest_branch, force, show_comments, False, @@ -176,7 +179,8 @@ def do_series(args, test_db=None, pwork=None, cser=None): cser, pwork, ups, args.patchwork_url) elif pwork and pwork is not True: raise ValueError( - f"Internal error: command '{args.subcmd}' should not have patchwork") + "Internal error: command " + f"'{args.subcmd}' should not have patchwork") if args.subcmd == 'add': cser.add(args.series, args.desc, mark=args.mark, allow_unmarked=args.allow_unmarked, end=args.upstream,