From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp51.i.mail.ru (smtp51.i.mail.ru [94.100.177.111]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id F34BF469719 for ; Wed, 14 Oct 2020 16:11:24 +0300 (MSK) References: <2a19304adfd3877d0fa26cde1d8d8c313a291588.1602677553.git.artemreyt@tarantool.org> From: Leonid Vasiliev Message-ID: <4efb7591-5d3e-d084-7899-f823416e1392@tarantool.org> Date: Wed, 14 Oct 2020 16:11:14 +0300 MIME-Version: 1.0 In-Reply-To: <2a19304adfd3877d0fa26cde1d8d8c313a291588.1602677553.git.artemreyt@tarantool.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Tarantool-patches] [PATCH] luacheck: fixed warning in tarantoolctl.in List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Artem Starshov , Alexander Turenko Cc: tarantool-patches@dev.tarantool.org LGTM. On 14.10.2020 15:14, Artem Starshov wrote: > Changed passing global variable arg to function > find_instance_name(arg) instead of passing arg[0] and arg[2] separately. > And removed exception in .luacheckrc for file /extra/dist/tarantoolctl.in. > > This change only solves linter warning, nothing else. > > Brach: https://github.com/tarantool/tarantool/tree/artemreyt/gh-4929-luacheck-warning-tarantoolctl > Issue: https://github.com/tarantool/tarantool/issues/4929 Branch and Issue are not a part of the commit message, add it after "---". (Don't resend this patch. This comment is for future patches) > > Fixed #4929. > ---