diff options
| author | Daniel Hahler <git@thequod.de> | 2019-07-04 05:30:16 +0200 |
|---|---|---|
| committer | Daniel Hahler <git@thequod.de> | 2019-07-04 12:29:54 +0200 |
| commit | 32361a1245d1584ae074702aa645bc20e353f787 (patch) | |
| tree | 66043fdb5d8e280f85de2552bbb72cc700e612f0 /test | |
| parent | 571b2c5e7be105d23ada987a7c7080f8631bfd15 (diff) | |
| download | rneovim-32361a1245d1584ae074702aa645bc20e353f787.tar.gz rneovim-32361a1245d1584ae074702aa645bc20e353f787.tar.bz2 rneovim-32361a1245d1584ae074702aa645bc20e353f787.zip | |
Improve luacheck setup [skip appveyor]
- Move .luacheckrc to root, add read_globals=vim
- Simplify lualint target, run it on all lua files
- Lint preload.lua, but ignore W211
- Remove testlint target, included in lualint (and lint)
- Clean up .luacheckrc
Diffstat (limited to 'test')
| -rw-r--r-- | test/.luacheckrc | 20 | ||||
| -rw-r--r-- | test/README.md | 4 |
2 files changed, 1 insertions, 23 deletions
diff --git a/test/.luacheckrc b/test/.luacheckrc deleted file mode 100644 index abfa881754..0000000000 --- a/test/.luacheckrc +++ /dev/null @@ -1,20 +0,0 @@ --- vim: ft=lua tw=80 - --- Don't report globals from luajit or busted (e.g. jit.os or describe). -std = '+luajit +busted' - --- One can't test these files properly; assume correctness. -exclude_files = { '*/preload.lua' } - --- Don't report unused self arguments of methods. -self = false - --- Rerun tests only if their modification time changed. -cache = true - -ignore = { - "631", -- max_line_length -} - --- Ignore whitespace issues in converted Vim legacy tests. ---files["functional/legacy"] = {ignore = { "611", "612", "613", "621" }} diff --git a/test/README.md b/test/README.md index c87f835f79..903e93e495 100644 --- a/test/README.md +++ b/test/README.md @@ -208,8 +208,6 @@ Guidelines [contained in an `it()` block](https://github.com/neovim/neovim/blob/d21690a66e7eb5ebef18046c7a79ef898966d786/test/functional/ex_cmds/grep_spec.lua#L11). Provide empty function argument if the `pending()` call is outside of `it()` ([example](https://github.com/neovim/neovim/commit/5c1dc0fbe7388528875aff9d7b5055ad718014de#diff-bf80b24c724b0004e8418102f68b0679R18)). -- Use `make testlint` for using the shipped luacheck program ([supported by syntastic](https://github.com/scrooloose/syntastic/blob/d6b96c079be137c83009827b543a83aa113cc011/doc/syntastic-checkers.txt#L3546)) - to lint all tests. - Really long `source([=[...]=])` blocks may break syntax highlighting. Try `:syntax sync fromstart` to fix it. @@ -235,7 +233,7 @@ by the semantic component they are testing. Lint ==== -`make lint` (and `make testlint`) runs [luacheck](https://github.com/mpeterv/luacheck) +`make lint` (and `make lualint`) runs [luacheck](https://github.com/mpeterv/luacheck) on the test code. If a luacheck warning must be ignored, specify the warning code. Example: |