On Wed, Jan 10, 2024 at 11:54 AM Aaron Conole <aconole@redhat.com> wrote:

> Adam Hassick <ahassick@iol.unh.edu> writes:
>
> > This script takes in a URL to a series on Patchwork and emits a
> > tarball which may be used for running tests.
> >
> > Signed-off-by: Adam Hassick <ahassick@iol.unh.edu>
> > ---
>
> Just a quick glance.
>
> Is it possible to include a doc on how it is expected to run this
> script?  Maybe that could help to also evaluate it as well.
>

I was thinking along similar lines when I ran the script, but to a certain
extent it seemed like having helpful examples in the config file would
clear up confusion. Nevertheless, I think it's certainly worth writing the
process down a little more. I wonder should we instead follow the example
of pw_maintainers_cli.py and have the example usage, how to run, etc.
written as a string at the top of the file? Then we are consistent with how
other scripts in the repo are operating.