From: Sergey Bronnikov via Tarantool-patches <tarantool-patches@dev.tarantool.org>
To: tarantool-patches@dev.tarantool.org, v.shpilevoy@tarantool.org
Subject: [Tarantool-patches] [PATCH v7 0/2] Fix luacheck warnings in test/sql and test/sql-tap
Date: Thu, 14 Jan 2021 11:52:19 +0300 [thread overview]
Message-ID: <cover.1610613097.git.sergeyb@tarantool.org> (raw)
From: Sergey Bronnikov <sergeyb@tarantool.org>
Changelog v7:
- fixed warnings in a new code
- rebased to master branch
Changelog v6:
- splitted patch in test/ for patches per sub-directory
- adjusted suppressions in .luacheckrc
- fixed formatting issues in .luacheckrc
Issues:
- https://github.com/tarantool/tarantool/issues/5463
- https://github.com/tarantool/tarantool/issues/5464
Gitlab CI: https://gitlab.com/tarantool/tarantool/-/pipelines/241203686
Branch: ligurio/gh-5463-luacheck-warnings-test-sql
Sergey Bronnikov (2):
test: fix luacheck warnings in test/sql
test: fix luacheck warnings in test/sql-tap
.luacheckrc | 2 +-
test/sql-tap/analyze9.test.lua | 18 +++++++++---------
test/sql-tap/between.test.lua | 4 ++--
test/sql-tap/date.test.lua | 3 ++-
test/sql-tap/e_expr.test.lua | 11 ++++++-----
test/sql-tap/func.test.lua | 2 +-
test/sql-tap/gh-2723-concurrency.test.lua | 8 ++++----
.../gh-3083-ephemeral-unref-tuples.test.lua | 2 +-
.../sql-tap/gh-3332-tuple-format-leak.test.lua | 2 +-
.../gh2127-indentifier-max-length.test.lua | 10 +++++-----
test/sql-tap/identifier-characters.test.lua | 2 +-
test/sql-tap/index1.test.lua | 4 +---
test/sql-tap/join3.test.lua | 2 +-
test/sql-tap/lua-tables.test.lua | 5 ++---
test/sql-tap/lua/sqltester.lua | 5 ++---
test/sql-tap/misc1.test.lua | 4 ++--
test/sql-tap/misc5.test.lua | 2 +-
test/sql-tap/select1.test.lua | 6 +++---
test/sql-tap/select2.test.lua | 8 ++++----
test/sql-tap/select4.test.lua | 3 +--
test/sql-tap/select5.test.lua | 3 +--
test/sql-tap/select9.test.lua | 10 +++-------
test/sql-tap/selectA.test.lua | 8 +-------
test/sql-tap/selectB.test.lua | 14 +++++++-------
test/sql-tap/selectG.test.lua | 1 -
test/sql-tap/sort.test.lua | 2 +-
test/sql-tap/sql-errors.test.lua | 5 ++---
test/sql-tap/table.test.lua | 3 ++-
test/sql-tap/tkt-38cb5df375.test.lua | 1 -
test/sql-tap/tkt-91e2e8ba6f.test.lua | 3 ---
test/sql-tap/tkt-9a8b09f8e6.test.lua | 3 ---
test/sql-tap/tkt-bd484a090c.test.lua | 3 ++-
test/sql-tap/tkt2192.test.lua | 3 ++-
test/sql-tap/tkt3493.test.lua | 3 ---
test/sql-tap/trigger2.test.lua | 4 ++--
test/sql-tap/triggerA.test.lua | 1 -
test/sql-tap/where2.test.lua | 6 ++----
test/sql-tap/whereC.test.lua | 1 -
test/sql-tap/with1.test.lua | 8 ++++----
test/sql-tap/with2.test.lua | 14 +++++++-------
test/sql/lua/sql_tokenizer.lua | 2 +-
41 files changed, 87 insertions(+), 114 deletions(-)
--
2.25.1
next reply other threads:[~2021-01-14 8:52 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-01-14 8:52 Sergey Bronnikov via Tarantool-patches [this message]
2021-01-14 8:52 ` [Tarantool-patches] [PATCH v7 1/2] test: fix luacheck warnings in test/sql Sergey Bronnikov via Tarantool-patches
2021-01-14 8:52 ` [Tarantool-patches] [PATCH v7 2/2] test: fix luacheck warnings in test/sql-tap Sergey Bronnikov via Tarantool-patches
2021-01-15 22:10 ` Vladislav Shpilevoy via Tarantool-patches
2021-01-21 12:52 ` Sergey Bronnikov via Tarantool-patches
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1610613097.git.sergeyb@tarantool.org \
--to=tarantool-patches@dev.tarantool.org \
--cc=sergeyb@tarantool.org \
--cc=v.shpilevoy@tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH v7 0/2] Fix luacheck warnings in test/sql and test/sql-tap' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox