[Tarantool-patches] [PATCH v2 07/10] test: backport lua-Harness directory detection
Максим Корякшин
m.kokryashkin at tarantool.org
Mon Jul 26 14:13:39 MSK 2021
Thanks for the review, Igor!
New commit message, considering all your comments:
=============================================
test: backport lua-Harness directory detection
The patch[1] from lua-Harness fixes the same issue commit
789820aac7cecc3b5ae394aadc12a5ec25b68121 ('test: fix directory detection
in lua-Harness suite') does, but it also works on Windows, so the
mentioned commit can be superseded by that patch.
[1]: https://framagit.org/fperrad/lua-Harness/-/commit/24a570c1
Part of tarantool/tarantool#5970
Part of tarantool/tarantool#4473
=============================================
>
>Понедельник, 26 июля 2021, 0:35 +03:00 от Igor Munkin <imun at tarantool.org>:
>
>Max,
>
>Thanks for the patch! LGTM, except the three typos.
>
>On 20.07.21, Maxim Kokryashkin wrote:
>> The patch[1] from lua-Harness fixes the same issue commit
>
>Typo: The link is missing.
>
>> 789820aac7cecc3b5ae394aadc12a5ec25b68121('test: fix directory detection
>
>Typo: Space is missing after the commit hash.
>
>> in lua-Harness suite') does, but it also works on Windows, so the
>> mentioned commit can be supreseded by that patch.
>
>Typo: s/supreseded/superseded/.
>
>>
>> Part of tarantool/tarantool#5970
>> Part of tarantool/tarantool#4473
>> ---
>> test/lua-Harness-tests/314-regex.t | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>
><snipped>
>
>> --
>> 2.32.0
>>
>
>--
>Best regards,
>IM
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20210726/77fa89e1/attachment.htm>
More information about the Tarantool-patches
mailing list