From fc6c663af38d2aa2c2438e4bc20227cdb3183e44 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Wed, 2 Mar 2022 19:12:24 -0700 Subject: [PATCH] patman: Define Commit.path in the constructor It is good practice to init all variables in the constructor and pylint sometimes checks this. Fix it. Signed-off-by: Simon Glass Reported-by: Tom Rini --- tools/patman/commit.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/patman/commit.py b/tools/patman/commit.py index a645b22d08..9537de43d3 100644 --- a/tools/patman/commit.py +++ b/tools/patman/commit.py @@ -28,6 +28,7 @@ class Commit: key: rtag type (e.g. 'Reviewed-by') value: Set of people who gave that rtag, each a name/email string warn: List of warnings for this commit, each a str + patch (str): Filename of the patch file for this commit """ def __init__(self, hash): self.hash = hash @@ -40,6 +41,7 @@ class Commit: self.change_id = None self.rtags = collections.defaultdict(set) self.warn = [] + self.patch = '' def __str__(self): return self.subject