[Tarantool-patches] [PATCH v2 luajit 02/45] test: actualize <LuaJIT-tests/README.md>

Sergey Kaplun skaplun at tarantool.org
Wed Aug 21 19:40:44 MSK 2024


Hi, Sergey!
Thanks for the review!
Please, consider my answers below.

On 21.08.24, Sergey Bronnikov wrote:
> Sergey,
> 
> On 21.08.2024 11:58, Sergey Kaplun wrote:
> > This patch syncs up the description of the LuaJIT-tests suite with the
> > current status. Also, it adds a description of the subcategories of
> > tests on which future patches will be based.
> >
> > Relates to tarantool/tarantool#9398
> > ---
> 
> thanks for the patch! LGTM
> 
> I would move this commit to the of patch series. Otherwise, "current 
> status" is confused,
> 
> because it is not current, actually.

I suppose it shouldn't be confusing, since the misc/ directory is
mentioned in the next section (as tests that are not enabled). And its
mention is deleted in the last commit of the series.

Should I rephrase the commit message somehow?

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list