[Tarantool-patches] [PATCH v8 14/14] luacheck: add issues for suppressed warnings
Sergey Bronnikov
sergeyb at tarantool.org
Fri Jan 29 19:13:38 MSK 2021
Thanks for review!
On 24.01.2021 20:37, Vladislav Shpilevoy wrote:
> Thanks for the patch!
>
> On 21.01.2021 13:50, sergeyb at tarantool.org wrote:
>> From: Sergey Bronnikov <sergeyb at tarantool.org>
>>
>> Closes #5464
>> ---
>> .luacheckrc | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/.luacheckrc b/.luacheckrc
>> index a798ca6f8..597988367 100644
>> --- a/.luacheckrc
>> +++ b/.luacheckrc
>> @@ -52,8 +52,10 @@ exclude_files = {
>> files["test/sql-tap/**/*.lua"] = {
>> ignore = {
>> -- A line consists of nothing but whitespace.
>> + -- https://github.com/tarantool/tarantool/issues/5739
> Why can't you fix this one now? Is it so hard that it is
> worth adding a special issue for it? Doesn't the whole work
> consists of removing a bunch of empty lines or whitespaces
> from these lines?
I can. luacheck reported about 1k warnings of this type
and I was worried that it would be difficult to review.
Added new commit with fixes for W611.
>> "611",
>> -- Line is too long.
>> + -- https://github.com/tarantool/tarantool/issues/5181
>> "631"
>> }
>> }
>>
More information about the Tarantool-patches
mailing list