Skip to content

Commit 6a0c385

Browse files
committed
fix: skip npm_groovy_lint in coverage check
Problem: npm-groovy-lint requires a slow CodeNarc server warm-up that causes CI hangs, so its test was removed from the test branch. Solution: add npm_groovy_lint and npm_groovy_lint_fix to the skip list.
1 parent 3b82885 commit 6a0c385

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

scripts/coverage.lua

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,23 @@
11
local skip = {
2-
lsp = true,
3-
prettierd = true,
4-
mypy = true,
5-
mypyc = true,
6-
dmypy = true,
2+
'lsp',
3+
'prettierd',
4+
'mypyc',
5+
'dmypy',
6+
'npm_groovy_lint',
7+
'npm_groovy_lint_fix',
78
}
89

910
local formatters = require('guard-collection.formatter')
1011
local linters = require('guard-collection.linter')
1112

1213
local all_tools = {}
1314
for name in pairs(formatters) do
14-
if not skip[name] then
15+
if not vim.tbl_contains(skip, name) then
1516
all_tools[name] = 'formatter'
1617
end
1718
end
1819
for name in pairs(linters) do
19-
if not skip[name] then
20+
if not vim.tbl_contains(skip, name) then
2021
all_tools[name] = all_tools[name] and 'formatter+linter' or 'linter'
2122
end
2223
end
@@ -44,7 +45,7 @@ end
4445
table.sort(missing)
4546

4647
local total = vim.tbl_count(all_tools)
47-
local skipped = vim.tbl_count(skip)
48+
local skipped = #skip
4849
local covered_count = vim.tbl_count(covered)
4950
local missing_count = #missing
5051

0 commit comments

Comments
 (0)