diff options
| author | Justin M. Keyes <justinkz@gmail.com> | 2023-12-07 12:19:35 +0100 |
|---|---|---|
| committer | Justin M. Keyes <justinkz@gmail.com> | 2024-01-03 02:09:28 +0100 |
| commit | 59d117ec99b6037cb9fad5bbfb6d0b18f5012927 (patch) | |
| tree | ac8f9b52013066f1145215dd46327d814b8cf02b /.styluaignore | |
| parent | b3eda5e73f65092d50ccd27a0373e8b9fad076b0 (diff) | |
| download | rneovim-59d117ec99b6037cb9fad5bbfb6d0b18f5012927.tar.gz rneovim-59d117ec99b6037cb9fad5bbfb6d0b18f5012927.tar.bz2 rneovim-59d117ec99b6037cb9fad5bbfb6d0b18f5012927.zip | |
build: enable lintlua for test/ dir
Problem:
Not all Lua code is checked by stylua. Automating code-style is an
important mechanism for reducing time spent on accidental
(non-essential) complexity.
Solution:
- Enable stylua for entire `test/` directory.
- Exclude these high-churn files until this issue is resolved: https://github.com/JohnnyMorganz/StyLua/issues/829
```
test/functional/ui/decorations_spec.lua | 3560 ++++++++++++++++++++++++++++++++++++----------------
test/functional/ui/float_spec.lua | 5826 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------
test/functional/ui/multigrid_spec.lua | 1349 ++++++++++++++------
```
- Make surgical changes to these files (or add `stylua: ignore` in some
small scopes) to improve the result:
```
test/functional/vimscript/msgpack_functions_spec.lua | 1414 +++++++++++++++------
test/functional/api/buffer_spec.lua | 1389 +++++++++++----------
test/functional/api/vim_spec.lua | 2740 +++++++++++++++++++++++-----------------
```
- These "high churn" files are NOT excluded because the changes are
largely an improvement:
```
test/functional/plugin/lsp_spec.lua | 2198 ++++++++++++++++++---------------
test/functional/plugin/shada_spec.lua | 4078 +++++++++++++++++++++++++++++++++++-------------------------
test/functional/ui/cmdline_spec.lua | 1199 +++++++++++-------
test/functional/ui/popupmenu_spec.lua | 1267 +++++++++++--------
test/functional/ui/messages_spec.lua | 1643 +++++++++++++++---------
```
- TODO: how to check "all directories"? With `GLOB_DIRS *` and `/.deps/` (or
`.deps/`) in `.styluaignore`, Lua code in `.deps/` is still checked...
Diffstat (limited to '.styluaignore')
| -rw-r--r-- | .styluaignore | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/.styluaignore b/.styluaignore index 53f7ea2812..80c6a9692c 100644 --- a/.styluaignore +++ b/.styluaignore @@ -1,9 +1,12 @@ -/build +/build/ +/.deps/ /runtime/lua/coxpcall.lua /runtime/lua/vim/_meta /runtime/lua/vim/re.lua -/test/functional +test/functional/ui/decorations_spec.lua +test/functional/ui/float_spec.lua +test/functional/ui/multigrid_spec.lua /test/functional/fixtures/lua/syntax_error.lua /test/functional/legacy/030_fileformats_spec.lua /test/functional/legacy/044_099_regexp_multibyte_magic_spec.lua |