Skip to content

Commit e04d0af

Browse files
committed
ci: exit codes with coverage now work
1 parent 6b0fdf2 commit e04d0af

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

.github/workflows/main.yaml

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,23 +25,16 @@ jobs:
2525

2626
- name: Installing Extension
2727
run: npm ci
28-
- name: Compile
29-
run: npm run compile
28+
- name: Compile (Development Build)
29+
run: npm run pretest
3030
- name: Linting
3131
run: npm run lint
3232
- name: Test Syntax Highlighting
3333
run: npm run test:grammar
34-
- name: Test Unittests
35-
if: runner.os == 'Linux'
36-
run: xvfb-run --auto-servernum npm run test
37-
- name: Test Unittests on Windows
38-
if: runner.os == 'Windows'
39-
run: npm run test
40-
# This will not fail the job if tests fail so we have to npm test separately
41-
- name: Coverage report
34+
- name: Run Unit and Integration Tests
4235
if: runner.os == 'Linux'
4336
run: xvfb-run --auto-servernum npm run coverage
44-
- name: Coverage report on Windows
37+
- name: Run Unit and Integration on Windows
4538
if: runner.os == 'Windows'
4639
run: npm run coverage
4740
- name: Upload coverage to Codecov

0 commit comments

Comments
 (0)