Skip to content

Commit 156e73d

Browse files
committed
Add coverage.mjs comments
1 parent 4544adf commit 156e73d

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

.github/scripts/coverage.mjs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@
22

33
const expected = 90
44
const exclude = [
5-
'expr/test',
6-
'checker/mock',
7-
'vm/func_types',
8-
'vm/runtime/helpers',
9-
'internal/difflib',
10-
'internal/spew',
11-
'internal/testify',
12-
'pro',
5+
'expr/test', // We do not need to test the test package.
6+
'checker/mock', // Mocks only used for testing.
7+
'vm/func_types', // Generated files.
8+
'vm/runtime/helpers', // Generated files.
9+
'internal/difflib', // Test dependency. This is vendored dependency, and ideally we also have good tests for it.
10+
'internal/spew', // Test dependency.
11+
'internal/testify', // Test dependency.
12+
'patcher/value', // Contains a lot of repeating code. Ideally we should have a test for it.
13+
'pro', // Expr Pro is not a part of the main codebase.
1314
]
1415

1516
cd(path.resolve(__dirname, '..', '..'))

0 commit comments

Comments
 (0)