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

[Sergey Bronnikov <sergeyb@tarantool.org>]

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@tarantool.org wrote:
> > From: Sergey Bronnikov <sergeyb@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@