[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