[Tarantool-patches] [PATCH v3 0/6] Add static analysis with luacheck
Sergey Bronnikov
sergeyb at tarantool.org
Tue Apr 14 10:55:48 MSK 2020
Changelog:
- removed asserts in test/ to keep following diff-based approach with tests
- added separate install target in .travis.mk
- added comment regarding ENABLE_DIST in .travis.mk
- enabled app-tap in .luacheckrc
- removed duplicate warnings W212 and W122
- disabled checking of files in test/var directory
GH issue: https://github.com/tarantool/tarantool/issues/4681
GitHub branch: https://github.com/tarantool/tarantool/tree/ligurio/gh-4681-fix-luacheck-warnings
Sergey Bronnikov (6):
Fix luacheck warnings in src/lua/
Fix luacheck warnings in test/
Fix luacheck warnings in src/box/lua/
Fix luacheck warnings in extra/dist/tarantoolctl.in
Add luacheck config
gitlab-ci: enable static analysis with luacheck
.gitlab-ci.yml | 9 +
.luacheckrc | 175 ++++++++++++++++++
.travis.mk | 15 +-
extra/dist/tarantoolctl.in | 18 +-
src/box/lua/upgrade.lua | 12 +-
src/lua/argparse.lua | 6 +-
src/lua/buffer.lua | 4 +-
src/lua/clock.lua | 2 +-
src/lua/crypto.lua | 4 +-
src/lua/csv.lua | 5 +-
src/lua/digest.lua | 2 +-
src/lua/env.lua | 2 +-
src/lua/fio.lua | 25 +--
src/lua/httpc.lua | 3 -
src/lua/init.lua | 7 +-
src/lua/msgpackffi.lua | 16 +-
src/lua/socket.lua | 65 ++++---
src/lua/string.lua | 1 -
src/lua/swim.lua | 10 +-
src/lua/tap.lua | 74 ++++----
src/lua/trigger.lua | 3 -
test/app-tap/console.test.lua | 4 -
test/app-tap/csv.test.lua | 28 +--
test/app-tap/logger.test.lua | 2 +-
test/box-py/box.lua | 2 +-
test/box-tap/gc.test.lua | 4 +-
test/box/box.lua | 2 +-
test/box/hash_multipart.result | 2 +-
test/box/hash_multipart.test.lua | 2 +-
test/box/lua/cfg_bad_vinyl_dir.lua | 2 +-
test/box/lua/cfg_rtree.lua | 2 +-
test/box/lua/cfg_test1.lua | 2 +-
test/box/lua/cfg_test2.lua | 2 +-
test/box/lua/cfg_test3.lua | 2 +-
test/box/lua/cfg_test4.lua | 2 +-
test/box/lua/cfg_test5.lua | 4 +-
test/box/lua/cfg_test6.lua | 2 +-
test/box/lua/utils.lua | 3 +-
test/box/on_schema_init.lua | 2 +-
test/box/proxy.lua | 2 +-
test/box/tiny.lua | 2 +-
test/box/tree_pk.result | 4 +-
test/box/tree_pk.test.lua | 4 +-
test/engine/tree_min_max_count.result | 2 +-
test/engine/tree_min_max_count.test.lua | 2 +-
test/replication-py/master.lua | 2 +-
test/replication-py/panic.lua | 2 +-
test/replication/master.lua | 2 +-
test/replication/replicaset_ro_mostly.result | 2 +-
.../replication/replicaset_ro_mostly.test.lua | 2 +-
test/sql-tap/analyze3.test.lua | 6 +-
test/sql-tap/analyze9.test.lua | 16 +-
.../gh-4077-iproto-execute-no-bind.test.lua | 7 +-
test/sql-tap/index1.test.lua | 1 -
test/sql-tap/join3.test.lua | 2 +-
test/sql-tap/select9.test.lua | 10 +-
test/sql-tap/tkt-fa7bf5ec.test.lua | 6 +-
test/sql-tap/where2.test.lua | 2 +-
test/sql/lua/sql_tokenizer.lua | 2 +-
test/sql/savepoints.result | 6 +-
test/sql/savepoints.test.lua | 6 +-
test/sql/triggers.result | 2 +-
test/sql/triggers.test.lua | 2 +-
test/vinyl/large.lua | 3 +-
test/vinyl/txn_proxy.lua | 6 +-
test/vinyl/vinyl.lua | 2 +-
test/wal_off/rtree_benchmark.result | 2 +-
test/wal_off/rtree_benchmark.test.lua | 2 +-
test/xlog-py/box.lua | 2 +-
test/xlog/panic.lua | 2 +-
test/xlog/reader.result | 2 +-
test/xlog/reader.test.lua | 2 +-
test/xlog/snap_io_rate.test.lua | 2 +-
test/xlog/transaction.result | 2 +-
test/xlog/transaction.test.lua | 2 +-
test/xlog/xlog.lua | 2 +-
76 files changed, 409 insertions(+), 243 deletions(-)
create mode 100644 .luacheckrc
--
2.23.0
--
sergeyb@
More information about the Tarantool-patches
mailing list