[PATCH 1/2] tools: Add script to create artifacts
Patrick Robb
probb at iol.unh.edu
Wed Jan 10 19:56:17 CET 2024
On Wed, Jan 10, 2024 at 11:54 AM Aaron Conole <aconole at redhat.com> wrote:
> Adam Hassick <ahassick at 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 at 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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/ci/attachments/20240110/d3e9d65b/attachment.htm>
More information about the ci
mailing list