[Tarantool-patches] [PATCH v5 04/10] Fix luacheck warnings in src/lua/

Sergey Bronnikov sergeyb at tarantool.org
Thu May 28 15:22:21 MSK 2020


Igor,

I have added globals and committed to branch.

On 14:22 Wed 27 May , Igor Munkin wrote:
> Sergey,
> 
> Thanks for the patch! I propose to add suppressions also for the
> remaining globals such as os, package in init.lua and help, tutorial in
> help.lua to fix all warnings reported for src/lua directory in scope of
> this patch.
> 
> Otherwise LGTM, as for the major patch.
> 
> On 12.05.20, sergeyb at tarantool.org wrote:
> > From: Sergey Bronnikov <sergeyb at tarantool.org>
> > 
> > Closes #4681
> > 
> > Reviewed-by: Vladislav Shpilevoy <v.shpilevoy at tarantool.org>
> > Reviewed-by: Igor Munkin <imun at tarantool.org>
> > 
> > Co-authored-by: Vladislav Shpilevoy <v.shpilevoy at tarantool.org>
> > Co-authored-by: Igor Munkin <imun at tarantool.org>
> > ---
> >  .luacheckrc | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/.luacheckrc b/.luacheckrc
> > index f39502063..3cd05a254 100644
> > --- a/.luacheckrc
> > +++ b/.luacheckrc
> > @@ -29,3 +29,6 @@ files["extra/dist/tarantoolctl.in"] = {
> >  	globals = {"box", "_TARANTOOL"},
> >  	ignore = {"212/self", "122", "431"}
> >  }
> > +files["src/lua/*.lua"] = {ignore = {"212/self"}}
> > +files["src/lua/init.lua"] = {globals = {"dostring"}}
> > +files["src/lua/swim.lua"] = {ignore = {"431"}}
> > -- 
> > 2.23.0
> > 
> 
> -- 
> Best regards,
> IM


More information about the Tarantool-patches mailing list