[Tarantool-patches] [PATCH v5 09/10] Add luacheck supressions for luajit tests

Sergey Bronnikov sergeyb at tarantool.org
Thu May 28 22:25:39 MSK 2020


Igor, thanks for review!

supression has been removed.

On 23:45 Tue 26 May , Igor Munkin wrote:
> Sergey,
> 
> Thanks for the patch! Since you've already fixed the warning <231> in
> test/luajit-tap/gh-4476-fix-string-find-recording.test.lua here[1], the
> corresponding suppression below can be removed.
>
> Furthermore, please bump LuaJIT submodule in scope of this patch, to
> pull the corresponding changes made in tarantool/luajit repo.
> 
> Otherwise, the patch LGTM, thanks!
> 
> On 12.05.20, sergeyb at tarantool.org wrote:
> > From: Sergey Bronnikov <sergeyb at tarantool.org>
> > 
> > Closes #4681
> > 
> > Reviewed-by: Igor Munkin <imun at tarantool.org>
> > ---
> >  .luacheckrc | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/.luacheckrc b/.luacheckrc
> > index 26986ffb1..62a0135d1 100644
> > --- a/.luacheckrc
> > +++ b/.luacheckrc
> > @@ -67,6 +67,8 @@ files["test/box/lua/fifo.lua"] = {globals = {"fifomax", "find_or_create_fifo", "
> >  files["test/box/lua/identifier.lua"] = {globals = {"run_test"}}
> >  files["test/box/lua/require_mod.lua"] = {globals = {"exports"}}
> >  files["test/engine/conflict.lua"] = {globals = {"test_conflict"}}
> > +files["test/luajit-tap/gh-4476-fix-string-find-recording.test.lua"] = {ignore = {"231"}}
> > +files["test/luajit-tap/or-232-unsink-64-kptr.test.lua"] = {ignore = {"542"}}
> >  files["test/replication/replica_quorum.lua"] = {globals = {"INSTANCE_URI", "nonexistent_uri"}}
> >  files["test/replication/replica_on_schema_init.lua"] = {globals = {"trig_local", "trig_engine"}}
> >  files["test/replication/lua/fast_replica.lua"] = {
> > -- 
> > 2.23.0
> > 
> 
> [1]: https://lists.tarantool.org/pipermail/tarantool-patches/2020-May/017066.html
> 
> -- 
> Best regards,
> IM

-- 
sergeyb@


More information about the Tarantool-patches mailing list