build: replace luacheck with selene
Problem: luacheck is unmaintained (last release 2018) and required suppressing four warning classes to avoid false positives. It also lacks first-class vim/neovim awareness. Solution: switch to selene with std='vim' for vim-aware linting. Replace the luacheck CI job with selene, update the Makefile lint target, and delete .luacheckrc.
This commit is contained in:
parent
0f8b084e4a
commit
364b787578
4 changed files with 7 additions and 32 deletions
17
.github/workflows/tests.yml
vendored
17
.github/workflows/tests.yml
vendored
|
|
@ -10,21 +10,14 @@ on:
|
||||||
- main
|
- main
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
luacheck:
|
selene:
|
||||||
name: Luacheck
|
name: Selene
|
||||||
runs-on: ubuntu-22.04
|
runs-on: ubuntu-22.04
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
|
- uses: NTBBloodbath/selene-action@v1.0.0
|
||||||
- name: Prepare
|
with:
|
||||||
run: |
|
args: --display-style quiet .
|
||||||
sudo apt-get update
|
|
||||||
sudo add-apt-repository universe
|
|
||||||
sudo apt install luarocks -y
|
|
||||||
sudo luarocks install luacheck
|
|
||||||
|
|
||||||
- name: Run Luacheck
|
|
||||||
run: luacheck lua tests
|
|
||||||
|
|
||||||
stylua:
|
stylua:
|
||||||
name: StyLua
|
name: StyLua
|
||||||
|
|
|
||||||
19
.luacheckrc
19
.luacheckrc
|
|
@ -1,19 +0,0 @@
|
||||||
max_comment_line_length = false
|
|
||||||
codes = true
|
|
||||||
|
|
||||||
exclude_files = {
|
|
||||||
"tests/treesitter",
|
|
||||||
}
|
|
||||||
|
|
||||||
ignore = {
|
|
||||||
"212", -- Unused argument
|
|
||||||
"631", -- Line is too long
|
|
||||||
"122", -- Setting a readonly global
|
|
||||||
"542", -- Empty if branch
|
|
||||||
}
|
|
||||||
|
|
||||||
read_globals = {
|
|
||||||
"vim",
|
|
||||||
"a",
|
|
||||||
"assert",
|
|
||||||
}
|
|
||||||
2
Makefile
2
Makefile
|
|
@ -17,7 +17,7 @@ test:
|
||||||
.PHONY: lint
|
.PHONY: lint
|
||||||
lint: scripts/nvim-typecheck-action
|
lint: scripts/nvim-typecheck-action
|
||||||
./scripts/nvim-typecheck-action/typecheck.sh --workdir scripts/nvim-typecheck-action lua
|
./scripts/nvim-typecheck-action/typecheck.sh --workdir scripts/nvim-typecheck-action lua
|
||||||
luacheck lua tests --formatter plain
|
selene --display-style quiet .
|
||||||
stylua --check lua tests
|
stylua --check lua tests
|
||||||
|
|
||||||
## profile: use LuaJIT profiler to profile the plugin
|
## profile: use LuaJIT profiler to profile the plugin
|
||||||
|
|
|
||||||
1
selene.toml
Normal file
1
selene.toml
Normal file
|
|
@ -0,0 +1 @@
|
||||||
|
std = 'vim'
|
||||||
Loading…
Add table
Add a link
Reference in a new issue