[Tarantool-patches] [PATCH v6 03/25] gitlab-ci: enable static analysis with luacheck

Sergey Bronnikov sergeyb at tarantool.org
Mon Jun 1 12:29:57 MSK 2020


Igor,

thanks for review!

On 22:25 Fri 29 May , Igor Munkin wrote:
> Sergey,
> 
> Thanks for the patch! It LGTM, except the one comment: I'm not a Gitlab
> CI expert but since you introduced a separate make rule for luacheck, it
> looks like you can use it also in CI. As a result of such change we have
> a single place where luacheck command is mentioned excplicitly in our
> project, that simplifies further maintenance.

You are right, it is a piece of duplicated code. But in a CI I don't use
tarantool build infrastructure at all, because we don't need to build
tarantool to check Lua code used in it. Let me know if you have an ideas
how to share luacheck command line with CMakeLists.txt and GitLab CI
config.

> On 29.05.20, sergeyb at tarantool.org wrote:
> > From: Sergey Bronnikov <sergeyb at tarantool.org>
> > 
> > Part of #4681
> > ---
> >  .gitlab-ci.yml | 11 +++++++++++
> >  .travis.mk     | 17 +++++++++++++++++
> >  2 files changed, 28 insertions(+)
> > 
> 
> <snipped>
> 
> > -- 
> > 2.23.0
> > 
> 
> -- 
> Best regards,
> IM

-- 
sergeyb@


More information about the Tarantool-patches mailing list