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.