ref: Refactor app state manager to not use protocol #1555
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Lint SwiftLint | |
| on: | |
| push: | |
| branches: | |
| - main | |
| pull_request: | |
| # Concurrency configuration: | |
| # - We use workflow-specific concurrency groups to prevent multiple lint runs on the same code, | |
| # as linting checks are deterministic and don't require parallel validation. | |
| # - For pull requests, we cancel in-progress runs when new commits are pushed since only the latest | |
| # linting matters for merge decisions. | |
| # - For main branch pushes, we never cancel formatting checks to ensure our code maintains consistent | |
| # style standards across the entire project. | |
| concurrency: | |
| group: ${{ github.workflow }}-${{ github.ref }} | |
| cancel-in-progress: ${{ github.event_name == 'pull_request' }} | |
| jobs: | |
| files-changed: | |
| name: Detect File Changes | |
| runs-on: ubuntu-latest | |
| outputs: | |
| run_lint_swiftlint_for_prs: ${{ steps.changes.outputs.run_lint_swiftlint_for_prs }} | |
| steps: | |
| - uses: actions/checkout@v5 | |
| - name: Get changed files | |
| id: changes | |
| uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2 | |
| with: | |
| token: ${{ github.token }} | |
| filters: .github/file-filters.yml | |
| lint: | |
| if: github.event_name != 'pull_request' || needs.files-changed.outputs.run_lint_swiftlint_for_prs == 'true' | |
| needs: files-changed | |
| name: Lint | |
| runs-on: macos-14 | |
| steps: | |
| - uses: actions/checkout@v5 | |
| - name: Install tooling | |
| run: make init-ci-format | |
| - run: swiftlint --version | |
| - name: Run SwiftLint | |
| run: swiftlint --strict | |
| - name: Run CI Diagnostics | |
| if: failure() | |
| run: ./scripts/ci-diagnostics.sh | |
| # This check validates that either lint passed or was skipped, which allows us | |
| # to make lint-swiftlint a required check with only running the lint when required. | |
| # So, we don't have to run lint-swiftlint, for example, for unrelated changes. | |
| lint_swiftlint-required-check: | |
| needs: | |
| [ | |
| files-changed, | |
| lint, | |
| ] | |
| name: Lint SwiftLint | |
| # This is necessary since a failed/skipped dependent job would cause this job to be skipped | |
| if: always() | |
| runs-on: ubuntu-latest | |
| steps: | |
| # If any jobs we depend on fails gets cancelled or times out, this job will fail. | |
| # Skipped jobs are not considered failures. | |
| - name: Check for failures | |
| if: contains(needs.*.result, 'failure') || contains(needs.*.result, 'cancelled') | |
| run: | | |
| echo "One of the lint-swiftlint jobs has failed." && exit 1 |