<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 3, 2023 at 2:33 PM Thomas Monjalon <<a href="mailto:thomas@monjalon.net">thomas@monjalon.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">03/04/2023 13:46, Juraj Linkeš:<br>
> Replace pexpect with Fabric.<br>
<br>
You should squash these lines with the move to Fabric.<br>
<br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> Signed-off-by: Juraj Linkeš <juraj.linkes@pantheon.tech><br>
> ---<br>
>  dts/poetry.lock    | 553 +++++++++++++++++++++++++++++++++++++++------<br>
<br>
Do we really need *all* these lines?<br>
I see a lot of lines about Windows and MacOSX which are not supported in DTS.<br>
It is so long that it looks impossible to review.<br>
<br></blockquote><div><br></div><div>This is a generated file and doesn't need to be reviewed. I separated the dependencies part so that the code part is easier to review. If you want, I can squash the two commits.<br></div></div></div>