Re: [Tarantool-patches] [PATCH v4 2/10] gitlab-ci: enable static analysis with luacheck

[Kirill Yukhin <kyukhin@tarantool.org>]

Hello,

On 21 апр 23:04, Alexander Tikhonov wrote:
> 
> Sergey, thank you for the patch, I have some questions on it.
> 
> Do we really want to have separate stage before the man stages with testing ? As I see you want to have the ability to avoid of testing if the initial luacheck test fails, but does it mean that issues found by the luacheck will always need changes in the code that always will change the rest of the jobs, I mean is it strong dependency between this check and the others that we can’t do the rest of the testing w/o luacheck ? We had the opinion that our testing should produce fails as fast as possible that is why we decided to remove different stages, and we can see the early fails as fast as possible, because currently all the testing start together from the very start, but in your fix will have more that 2+ minutes delay before it, while we had strong suggestion from Kirill Y. to make the testing fast as much as possible and try to make it parallel up to 2 minutes. Right, for now we have overall time of testing up to 20 minutes, but in general fastest jobs finishes in 1+ minutes and we can see the issues much faster than waiting the whole testing process.

Please use lines of sane length.

--
Regards, Kirill Yukhin