[PATCH 3/3] utils: checkstyle.py: Add __repr__ method to Commit class
Milan Zamazal
mzamazal at redhat.com
Mon Aug 12 08:45:45 CEST 2024
Laurent Pinchart <laurent.pinchart at ideasonboard.com> writes:
> When debugging issues with the Commit class, a __repr__ method proved to
> be useful to quickly print all the parsed information about a commit. To
> avoid reimplementing the method over and over again in the future, add
> it to the class, even if it is not actually used.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Milan Zamazal <mzamazal at redhat.com>
> ---
> utils/checkstyle.py | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/utils/checkstyle.py b/utils/checkstyle.py
> index 2b1e1f6c1b9e..c9e41d4149f7 100755
> --- a/utils/checkstyle.py
> +++ b/utils/checkstyle.py
> @@ -248,6 +248,17 @@ class Commit:
> stdout=subprocess.PIPE).stdout.decode('utf-8')
> self._files = [CommitFile(f) for f in ret.splitlines()]
>
> + def __repr__(self):
> + return '\n'.join([
> + f'commit {self.commit}',
> + f'Author: {self.author}',
> + f'',
> + f' {self.title}',
> + '',
> + '\n'.join([line and f' {line}' or '' for line in self._body]),
> + 'Trailers:',
> + ] + self.trailers)
> +
> def files(self, filter='AMR'):
> return [f.filename for f in self._files if f.status in filter]
More information about the libcamera-devel
mailing list