Skip to content

Commit

Permalink
fix(repo): GHA timeout-minutes conversion (#2035)
Browse files Browse the repository at this point in the history
  • Loading branch information
tmilewski authored Nov 3, 2023
1 parent 88faa3b commit efe6d06
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/base-e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
run:
continue-on-error: true
runs-on: ${{ vars.RUNNER_NORMAL }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_LONG }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_LONG) }}
concurrency:
group: integration-${{ github.ref }}-${{ inputs.script }}
cancel-in-progress: true
Expand Down
18 changes: 9 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,11 @@ jobs:
formatting-linting:
name: Formatting, linting & changeset checks
runs-on: ${{ vars.RUNNER_NORMAL }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}

steps:
- name: Checkout Repo
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
uses: actions/checkout@v4
with:
fetch-depth: 0
Expand All @@ -47,31 +47,31 @@ jobs:
echo "FORCE_COLOR: $FORCE_COLOR"
- name: Require Changeset
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
run: if [ "${{ github.event.pull_request.user.login }}" = "clerk-cookie" ]; then echo 'Skipping' && exit 0; else npx changeset status --since=origin/main; fi

- name: Check Formatting
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
run: npm run format:check

- name: Lint packages using publint
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
run: npm run lint:publint -- $TURBO_ARGS

- name: Lint types using attw
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
run: npm run lint:attw -- $TURBO_ARGS
continue-on-error: true # TODO: Remove this when all related errors are fixed

- name: Run lint
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_SHORT }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_SHORT) }}
run: npm run lint -- $TURBO_ARGS -- --quiet

unit-tests:
name: Unit Tests
needs: formatting-linting
runs-on: ${{ vars.RUNNER_LARGE }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}

strategy:
matrix:
Expand Down Expand Up @@ -102,7 +102,7 @@ jobs:
name: Integration Tests
needs: formatting-linting
runs-on: ${{ vars.RUNNER_LARGE }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}

strategy:
matrix:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-snapshot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
snapshot-release:
if: ${{ startsWith(github.event.comment.body, '!snapshot') && github.repository == 'clerk/javascript' && github.event.issue.pull_request }}
runs-on: ${{ vars.RUNNER_LARGE }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}
env:
TURBO_TOKEN: ${{ secrets.TURBO_TOKEN }}
TURBO_TEAM: ${{ vars.TURBO_TEAM }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-staging.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
staging-release:
if: ${{ github.repository == 'clerk/javascript' }}
runs-on: ${{ vars.RUNNER_NORMAL }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}
env:
TURBO_TOKEN: ${{ secrets.TURBO_TOKEN }}
TURBO_TEAM: ${{ vars.TURBO_TEAM }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
name: Release
if: ${{ github.repository == 'clerk/javascript' }}
runs-on: ${{ vars.RUNNER_NORMAL }}
timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL }}
timeout-minutes: ${{ fromJSON(vars.TIMEOUT_MINUTES_NORMAL) }}
permissions:
contents: write
id-token: write
Expand Down

0 comments on commit efe6d06

Please sign in to comment.