> -----Original Message-----
> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: Thursday, November 10, 2022 11:26 AM
> To: Juraj Linkeš <juraj.lin...@pantheon.tech>
> Cc: dev@dpdk.org; Lijuan Tu <lijuan...@intel.com>; Owen Hilyard
> <ohily...@iol.unh.edu>
> Subject: Re: [PATCH] devtools: set DTS directory to format check
>
> 10/11/2022 10:27, Juraj Linkeš:
> > From: Thomas Monjalon <tho...@monjalon.net>
> > > usage() {
> > > - echo "Run formatting and linting programs for DTS. Usage:"
> > > -
> > > + echo 'Usage: $(basename $0) [options] [directory]'
> >
> > Double quotes here, otherwise $0 won't be expanded.
>
> I wonder how I tested it :)
>
> > > +directory=${1:-$(dirname $0)/../dts} cd $directory || exit 1
> >
> > I'd like to include the information of where we're doing the fomatting in
> the console output, e.g.:
> > echo "Formatting in $(pwd):"
> >
> > We're silently chaning the directory, so this would be useful when running
> with no argument and the script doesn't change anything - as a confirmation
> that it ran over the files we wanted to.
>
> Good comment, I'll improve in v2.
One more thing, if we go with changing the Formatting echo, then we should also
change the Linting echo. Or we could do it in some other place just once, I'm
not sure which is better.
>
> > Other than that,
> > Reviewed-by: Juraj Linkeš <juraj.lin...@pantheon.tech>
> > Tested-by: Juraj Linkeš <juraj.lin...@pantheon.tech>
>
> No it does not work, so you should not add your Tested-by.
> And in general, Reviewed-by is enough.
> And really, give your reviewed-by only when it's perfect :)
>
Thanks. It actually does work (by which I mean it does format and lint), but
it's not perfect, so I guess Tested-by should be fine, but Reviewed-by isn't?
:-)
