Skip to content

Commit 2f25cb0

Browse files
committed
ci: add luacheck linter
There was a linter check in test in `testing` workflow, but not in a separate job. After the patch a linter check is available in a separate job and the previous step from `testing` workflow was deleted. Closes #21
1 parent 853fb92 commit 2f25cb0

File tree

2 files changed

+37
-3
lines changed

2 files changed

+37
-3
lines changed

.github/workflows/check.yaml

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
name: Run static analysis
2+
3+
on:
4+
push:
5+
pull_request:
6+
7+
jobs:
8+
run-static-analysis:
9+
if: |
10+
github.event_name == 'push' ||
11+
github.event_name == 'pull_request' &&
12+
github.event.pull_request.head.repo.full_name != github.repository
13+
14+
runs-on: ubuntu-22.04
15+
16+
steps:
17+
- uses: actions/checkout@master
18+
19+
- name: Setup Tarantool CE
20+
uses: tarantool/setup-tarantool@v3
21+
with:
22+
tarantool-version: '3.2.0'
23+
24+
- name: Setup tt
25+
run: |
26+
curl -L https://tarantool.io/release/3/installer.sh | sudo bash
27+
sudo apt install -y tt
28+
tt version
29+
30+
- name: Setup luacheck
31+
run: make deps depname=lint
32+
33+
- run: echo $PWD/.rocks/bin >> $GITHUB_PATH
34+
35+
- name: Run static analysis
36+
run: make check

.github/workflows/testing.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,13 +95,11 @@ jobs:
9595
tt version
9696
9797
- name: Install requirements
98-
run: make deps
98+
run: make deps depname=coverage
9999
if: steps.cache-rocks.outputs.cache-hit != 'true'
100100

101101
- run: echo $PWD/.rocks/bin >> $GITHUB_PATH
102102

103-
- run: make check
104-
105103
- run: make test
106104

107105
- name: Send code coverage to 'coveralls.io'

0 commit comments

Comments
 (0)