diff --git a/.github/workflows/linter.yml b/.github/workflows/linter.yml index f6aeba7d80..dcbb665de1 100644 --- a/.github/workflows/linter.yml +++ b/.github/workflows/linter.yml @@ -1,47 +1,48 @@ name: ESLint on: - pull_request: - branches: [master] + pull_request: + branches: [master] jobs: - lint: - name: Lint updated JavaScript files with ESLint - - runs-on: ubuntu-latest - - steps: - - name: Checkout code - uses: actions/checkout@v4 - with: - fetch-depth: 0 - ref: ${{ github.event.pull_request.head.sha }} - - - name: Set up Node.js - uses: actions/setup-node@v4 - with: - node-version: 20.x - - - name: Get changed JavaScript files - id: get_files - run: | - CHANGED_FILES=$(git diff --diff-filter=ACMRT --name-only ${{ github.event.pull_request.base.sha }}..${{ github.event.pull_request.head.sha }} -- '*.js') - echo "files<> $GITHUB_ENV - echo "$CHANGED_FILES" >> $GITHUB_ENV - echo "EOF" >> $GITHUB_ENV - - - name: Install dependencies - run: npm ci - - - name: Run ESLint on changed files - if: env.files != '' - run: | - echo "Linting the following files:" - echo "$files" - echo "$files" | xargs npx eslint - - name: Run Prettier check on changed files - if: env.files != '' - run: | - echo "Checking formatting for the following files:" - echo "$files" - echo "$files" | xargs npx prettier --check + lint: + name: Lint updated JavaScript files with ESLint + + runs-on: ubuntu-latest + + steps: + - name: Checkout code + uses: actions/checkout@v4 + with: + fetch-depth: 0 + ref: ${{ github.event.pull_request.head.sha }} + + - name: Set up Node.js + uses: actions/setup-node@v4 + with: + node-version: 20.x + cache: "npm" + + - name: Get changed JavaScript files + id: get_files + run: | + CHANGED_FILES=$(git diff --diff-filter=ACMRT --name-only ${{ github.event.pull_request.base.sha }}..${{ github.event.pull_request.head.sha }} -- '*.js') + echo "files<> $GITHUB_ENV + echo "$CHANGED_FILES" >> $GITHUB_ENV + echo "EOF" >> $GITHUB_ENV + + - name: Install dependencies + run: npm ci + + - name: Run ESLint on changed files + if: env.files != '' + run: | + echo "Linting the following files:" + echo "$files" + echo "$files" | xargs npx eslint + - name: Run Prettier check on changed files + if: env.files != '' + run: | + echo "Checking formatting for the following files:" + echo "$files" + echo "$files" | xargs npx prettier --check