ci: replace stylua action with our own lint target

This will prevent situations where the linting works on CI but not
locally, at the cost of increased CI time.

Also manually ignore `runtime/vim/lua/re.lua`, as the .styluaignore
isn't respected when specifying a file instead of a directory.
This commit is contained in:
dundargoc
2023-04-29 01:48:14 +02:00
committed by GitHub
parent 4720f4379a
commit 057af3203b
2 changed files with 11 additions and 6 deletions

View File

@@ -42,6 +42,13 @@ jobs:
- name: Install dependencies - name: Install dependencies
run: ./.github/scripts/install_deps.sh lua-check run: ./.github/scripts/install_deps.sh lua-check
- name: Set up Homebrew
id: homebrew
uses: Homebrew/actions/setup-homebrew@master
- run: |
brew install stylua
- name: Cache uncrustify - name: Cache uncrustify
id: cache-uncrustify id: cache-uncrustify
uses: actions/cache@v3 uses: actions/cache@v3
@@ -84,12 +91,8 @@ jobs:
run: echo "status=${{ job.status }}" >> $GITHUB_OUTPUT run: echo "status=${{ job.status }}" >> $GITHUB_OUTPUT
- if: success() || failure() && steps.abort_job.outputs.status == 'success' - if: success() || failure() && steps.abort_job.outputs.status == 'success'
name: lintstylua name: stylua
uses: JohnnyMorganz/stylua-action@v2 run: cmake --build build --target lintlua-stylua
with:
token: ${{ secrets.GITHUB_TOKEN }}
version: latest
args: --check runtime/
- if: success() || failure() && steps.abort_job.outputs.status == 'success' - if: success() || failure() && steps.abort_job.outputs.status == 'success'
name: luacheck name: luacheck

View File

@@ -236,6 +236,8 @@ add_glob_target(
FLAGS --color=always --check FLAGS --color=always --check
GLOB_DIRS runtime/ GLOB_DIRS runtime/
GLOB_PAT *.lua GLOB_PAT *.lua
EXCLUDE
/runtime/lua/vim/re.lua
TOUCH_STRATEGY SINGLE) TOUCH_STRATEGY SINGLE)
add_custom_target(lintlua) add_custom_target(lintlua)