Skip to content

Commit

Permalink
ci: add luacheck linter
Browse files Browse the repository at this point in the history
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
  • Loading branch information
themilchenko committed Nov 5, 2024
1 parent f8d2c88 commit 9cf0640
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 3 deletions.
36 changes: 36 additions & 0 deletions .github/workflows/check.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
name: Run static analysis

on:
push:
pull_request:

jobs:
run-static-analysis:
if: |
github.event_name == 'push' ||
github.event_name == 'pull_request' &&
github.event.pull_request.head.repo.full_name != github.repository
runs-on: ubuntu-22.04

steps:
- uses: actions/checkout@master

- name: Setup Tarantool CE
uses: tarantool/setup-tarantool@v3
with:
tarantool-version: '3.2.0'

- name: Setup tt
run: |
curl -L https://tarantool.io/release/3/installer.sh | sudo bash
sudo apt install -y tt
tt version
- name: Setup luacheck
run: make deps depname=lint

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

- name: Run static analysis
run: make check
4 changes: 1 addition & 3 deletions .github/workflows/testing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -95,13 +95,11 @@ jobs:
tt version
- name: Install requirements
run: make deps
run: make deps depname=coverage
if: steps.cache-rocks.outputs.cache-hit != 'true'

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

- run: make check

- run: make test

- name: Send code coverage to 'coveralls.io'
Expand Down

0 comments on commit 9cf0640

Please sign in to comment.