From 057af3203b58771f07eb9c809573dd9275a597cb Mon Sep 17 00:00:00 2001 From: dundargoc <33953936+dundargoc@users.noreply.github.com> Date: Sat, 29 Apr 2023 01:48:14 +0200 Subject: 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. --- .github/workflows/test.yml | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) (limited to '.github/workflows') diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 2b0b9ff20a..0a44075f8e 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -42,6 +42,13 @@ jobs: - name: Install dependencies 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 id: cache-uncrustify uses: actions/cache@v3 @@ -84,12 +91,8 @@ jobs: run: echo "status=${{ job.status }}" >> $GITHUB_OUTPUT - if: success() || failure() && steps.abort_job.outputs.status == 'success' - name: lintstylua - uses: JohnnyMorganz/stylua-action@v2 - with: - token: ${{ secrets.GITHUB_TOKEN }} - version: latest - args: --check runtime/ + name: stylua + run: cmake --build build --target lintlua-stylua - if: success() || failure() && steps.abort_job.outputs.status == 'success' name: luacheck -- cgit