From b6cebe2c67ee6ec9ef996d862c126026c36e9b79 Mon Sep 17 00:00:00 2001 From: Jan Date: Tue, 26 May 2026 00:18:23 +0200 Subject: [PATCH] chore: linting in separate ci step (#814) * chore: lint ModMan UI in CI * chore: do not lint in build step anymore * chore: also do linting on main --- .github/workflows/ci.yaml | 2 -- .../{check-formatting.yaml => linting.yaml} | 33 ++++++++++++++++--- 2 files changed, 28 insertions(+), 7 deletions(-) rename .github/workflows/{check-formatting.yaml => linting.yaml} (51%) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index fe4d48de..f1785b59 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -68,7 +68,6 @@ jobs: working-directory: ${{ github.workspace }} run: | npm install - npm run lint npm run build - name: Build @@ -127,7 +126,6 @@ jobs: working-directory: ${{ github.workspace }} run: | npm install - npm run lint npm run build - name: Build diff --git a/.github/workflows/check-formatting.yaml b/.github/workflows/linting.yaml similarity index 51% rename from .github/workflows/check-formatting.yaml rename to .github/workflows/linting.yaml index f30e669b..da18631f 100644 --- a/.github/workflows/check-formatting.yaml +++ b/.github/workflows/linting.yaml @@ -1,11 +1,14 @@ -name: check-formatting +name: linting on: + push: + branches: + - main pull_request: jobs: - check-formatting: - name: Check formatting + check-clang-format-formatting: + name: Check clang-format formatting runs-on: ubuntu-latest container: ubuntu:24.04 defaults: @@ -19,8 +22,6 @@ jobs: - name: Checkout repository uses: actions/checkout@v6 - with: - submodules: recursive - name: Configure clang uses: ./.github/actions/configure-clang @@ -30,3 +31,25 @@ jobs: - name: Test formatting for all files working-directory: ${{ github.workspace }} run: ./scripts/check-format.sh + + modman-linting: + name: Lint ModMan UI + runs-on: ubuntu-latest + container: ubuntu:24.04 + defaults: + run: + shell: bash + steps: + - name: Checkout repository + uses: actions/checkout@v6 + + - name: Install node + uses: actions/setup-node@v6.4.0 + with: + node-version: 24 + + - name: Lint UI + working-directory: ${{ github.workspace }} + run: | + npm install + npm run lint