Re: [PATCH v7 1/3] check-uapi: Introduce check-uapi.sh
From: John Moon
Date: Wed Jan 10 2024 - 12:25:47 EST
On 12/22/2023 9:50 AM, Masahiro Yamada wrote:
On Tue, Dec 12, 2023 at 11:04 AM John Moon <quic_johmoo@xxxxxxxxxxx> wrote:
The code looks OK. I think it should work as designed.
Line 197 is inconsistently indented by spaces instead of a space,
but I can fix it up locally.
I just thought requiring target commits as positional parameters
("check-uapi.sh treeA treeB" just like "git diff treeA treeB")
might be intuitive, but I guess everything is specified
by a short option is a design. I can live with that.
I will wait a few days, and if there is nothing more,
I will pick up 1/3 and 2/3.
Thank you Masahiro! If anyone else has comments please let me know.
Cheers,
John