-
Notifications
You must be signed in to change notification settings - Fork 453
ci(repo): Improve CI/CD pipeline performance #7896
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 8 commits
461fc36
50b7c28
306a539
1a8c2a7
c064b44
755487d
c030ca6
83c31f1
bcaf79d
2d9edd8
0e5ca93
0ff83ba
d132529
6175673
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -69,10 +69,10 @@ jobs: | |||||||||||||||
| - name: Setup | ||||||||||||||||
| id: config | ||||||||||||||||
| uses: ./.github/actions/init-blacksmith | ||||||||||||||||
| # with: | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| with: | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Verify lockfile is deduped | ||||||||||||||||
| run: pnpm dedupe --check | ||||||||||||||||
|
|
@@ -117,11 +117,11 @@ jobs: | |||||||||||||||
| - name: Setup | ||||||||||||||||
| id: config | ||||||||||||||||
| uses: ./.github/actions/init-blacksmith | ||||||||||||||||
| # with: | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| with: | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Turbo Build | ||||||||||||||||
| run: pnpm turbo build $TURBO_ARGS --only | ||||||||||||||||
|
|
@@ -136,7 +136,7 @@ jobs: | |||||||||||||||
| retention-days: 5 | ||||||||||||||||
|
|
||||||||||||||||
| static-analysis: | ||||||||||||||||
| needs: [check-permissions, build-packages] | ||||||||||||||||
| needs: [check-permissions] | ||||||||||||||||
| if: ${{ github.event_name != 'pull_request' || github.event.pull_request.draft == false }} | ||||||||||||||||
| name: Static analysis | ||||||||||||||||
| permissions: | ||||||||||||||||
|
|
@@ -163,11 +163,11 @@ jobs: | |||||||||||||||
| - name: Setup | ||||||||||||||||
| id: config | ||||||||||||||||
| uses: ./.github/actions/init-blacksmith | ||||||||||||||||
| # with: | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| with: | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Check size using bundlewatch | ||||||||||||||||
| continue-on-error: true | ||||||||||||||||
|
|
@@ -199,7 +199,7 @@ jobs: | |||||||||||||||
| retention-days: 5 | ||||||||||||||||
|
|
||||||||||||||||
| unit-tests: | ||||||||||||||||
| needs: [check-permissions, build-packages] | ||||||||||||||||
| needs: [check-permissions] | ||||||||||||||||
| if: ${{ github.event_name != 'pull_request' || github.event.pull_request.draft == false }} | ||||||||||||||||
| name: Unit Tests (${{ matrix.filter-label }}) | ||||||||||||||||
| permissions: | ||||||||||||||||
|
|
@@ -237,10 +237,10 @@ jobs: | |||||||||||||||
| with: | ||||||||||||||||
| # Ensures that all builds are cached appropriately with a consistent run name `Unit Tests (20)`. | ||||||||||||||||
| node-version: ${{ matrix.node-version }} | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Run tests in packages | ||||||||||||||||
| run: | | ||||||||||||||||
|
|
@@ -272,10 +272,94 @@ jobs: | |||||||||||||||
| path: .turbo/runs | ||||||||||||||||
| retention-days: 5 | ||||||||||||||||
|
|
||||||||||||||||
| integration-tests: | ||||||||||||||||
| # needs: [check-permissions, build-packages] | ||||||||||||||||
| compute-affected-integration: | ||||||||||||||||
| needs: [check-permissions] | ||||||||||||||||
| if: ${{ github.event_name != 'pull_request' || github.event.pull_request.draft == false }} | ||||||||||||||||
| name: Compute Affected Integration Suites | ||||||||||||||||
| runs-on: "blacksmith-8vcpu-ubuntu-2204" | ||||||||||||||||
| defaults: | ||||||||||||||||
| run: | ||||||||||||||||
| shell: bash | ||||||||||||||||
| timeout-minutes: ${{ vars.TIMEOUT_MINUTES_NORMAL && fromJSON(vars.TIMEOUT_MINUTES_NORMAL) || 10 }} | ||||||||||||||||
| outputs: | ||||||||||||||||
| affected: ${{ steps.compute.outputs.affected }} | ||||||||||||||||
|
|
||||||||||||||||
| steps: | ||||||||||||||||
| - name: Checkout Repo | ||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||
| with: | ||||||||||||||||
| fetch-depth: 1 | ||||||||||||||||
| fetch-tags: false | ||||||||||||||||
| filter: "blob:none" | ||||||||||||||||
| show-progress: false | ||||||||||||||||
|
|
||||||||||||||||
| - name: Setup | ||||||||||||||||
| id: config | ||||||||||||||||
| uses: ./.github/actions/init-blacksmith | ||||||||||||||||
| with: | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Build packages | ||||||||||||||||
| run: pnpm turbo build $TURBO_ARGS --only | ||||||||||||||||
|
|
||||||||||||||||
| - name: Compute affected integration suites | ||||||||||||||||
| id: compute | ||||||||||||||||
| env: | ||||||||||||||||
| E2E_APP_CLERK_JS_DIR: ${{runner.temp}} | ||||||||||||||||
| E2E_APP_CLERK_UI_DIR: ${{runner.temp}} | ||||||||||||||||
| E2E_CLERK_JS_VERSION: "latest" | ||||||||||||||||
| E2E_CLERK_UI_VERSION: "latest" | ||||||||||||||||
| E2E_PROJECT: "chrome" | ||||||||||||||||
| run: | | ||||||||||||||||
| TEST_NAMES=( | ||||||||||||||||
| "generic" "express" "ap-flows" "localhost" | ||||||||||||||||
| "sessions" "sessions:staging" "handshake" "handshake:staging" | ||||||||||||||||
| "astro" "tanstack-react-start" "vue" "nuxt" | ||||||||||||||||
| "react-router" "custom" "billing" "machine" | ||||||||||||||||
| "nextjs" "quickstart" "cache-components" | ||||||||||||||||
| ) | ||||||||||||||||
|
|
||||||||||||||||
| # Check if frontend packages changed. Turbo integration tasks only | ||||||||||||||||
| # track integration/** as inputs, so they miss changes in packages | ||||||||||||||||
| # that all E2E tests depend on. When these packages change, force | ||||||||||||||||
| # all suites to run. | ||||||||||||||||
| FORCE_ALL=false | ||||||||||||||||
| BASE_REF="${{ github.event.pull_request.base.ref || 'main' }}" | ||||||||||||||||
| git fetch origin "${BASE_REF}" --depth=1 2>/dev/null || true | ||||||||||||||||
| CHANGED_FILES=$(git diff --name-only "origin/${BASE_REF}...HEAD" 2>/dev/null || true) | ||||||||||||||||
| if echo "$CHANGED_FILES" | grep -qE '^packages/(clerk-js|ui|react|shared|nextjs|remix|tanstack-react-start|vue|nuxt|astro|express|react-router|backend|types)/src/'; then | ||||||||||||||||
| echo "Frontend/SDK package source changes detected, forcing all suites" | ||||||||||||||||
| FORCE_ALL=true | ||||||||||||||||
| fi | ||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Silent git failures may cause unexpected behavior. If 🔧 Proposed fix to handle git failures more explicitly FORCE_ALL=false
BASE_REF="${{ github.event.pull_request.base.ref || 'main' }}"
- git fetch origin "${BASE_REF}" --depth=1 2>/dev/null || true
- CHANGED_FILES=$(git diff --name-only "origin/${BASE_REF}...HEAD" 2>/dev/null || true)
+ if ! git fetch origin "${BASE_REF}" --depth=1 2>/dev/null; then
+ echo "Warning: Could not fetch base ref ${BASE_REF}, forcing all suites"
+ FORCE_ALL=true
+ fi
+ if [ "$FORCE_ALL" != "true" ]; then
+ CHANGED_FILES=$(git diff --name-only "origin/${BASE_REF}...HEAD" 2>/dev/null || echo "")
+ if [ -z "$CHANGED_FILES" ] && [ "$(git rev-list --count origin/${BASE_REF}...HEAD 2>/dev/null || echo 0)" -gt 0 ]; then
+ echo "Warning: git diff returned empty unexpectedly, forcing all suites"
+ FORCE_ALL=true
+ fi
+ fi🤖 Prompt for AI Agents |
||||||||||||||||
|
|
||||||||||||||||
| AFFECTED_JSON="{" | ||||||||||||||||
| FIRST=true | ||||||||||||||||
| for name in "${TEST_NAMES[@]}"; do | ||||||||||||||||
| if [ "$FORCE_ALL" = "true" ]; then | ||||||||||||||||
| IS_AFFECTED=true | ||||||||||||||||
| else | ||||||||||||||||
| TASK_COUNT=$(pnpm turbo run "test:integration:${name}" --affected --dry=json 2>/dev/null | jq '.tasks | length' 2>/dev/null || echo "0") | ||||||||||||||||
| [ "$TASK_COUNT" -gt 0 ] && IS_AFFECTED=true || IS_AFFECTED=false | ||||||||||||||||
| fi | ||||||||||||||||
| $FIRST || AFFECTED_JSON+="," | ||||||||||||||||
| AFFECTED_JSON+="\"${name}\":${IS_AFFECTED}" | ||||||||||||||||
| if [ "$IS_AFFECTED" = "true" ]; then | ||||||||||||||||
| echo " affected: $name" | ||||||||||||||||
| else | ||||||||||||||||
| echo " skipped: $name" | ||||||||||||||||
| fi | ||||||||||||||||
| FIRST=false | ||||||||||||||||
| done | ||||||||||||||||
| AFFECTED_JSON+="}" | ||||||||||||||||
|
|
||||||||||||||||
| echo "affected=${AFFECTED_JSON}" >> $GITHUB_OUTPUT | ||||||||||||||||
| echo "Affected: ${AFFECTED_JSON}" | ||||||||||||||||
|
|
||||||||||||||||
| integration-tests: | ||||||||||||||||
| needs: [compute-affected-integration] | ||||||||||||||||
| if: ${{ github.event_name != 'pull_request' || github.event.pull_request.draft == false }} | ||||||||||||||||
| name: Integration Tests (${{ matrix.test-name }}, ${{ matrix.test-project }}${{ matrix.next-version && format(', {0}', matrix.next-version) || '' }}) | ||||||||||||||||
| permissions: | ||||||||||||||||
| contents: read | ||||||||||||||||
|
|
@@ -329,7 +413,18 @@ jobs: | |||||||||||||||
| next-version: "16" | ||||||||||||||||
|
|
||||||||||||||||
| steps: | ||||||||||||||||
| - name: Check if affected | ||||||||||||||||
| id: affected | ||||||||||||||||
| run: | | ||||||||||||||||
| AFFECTED_JSON='${{ needs.compute-affected-integration.outputs.affected }}' | ||||||||||||||||
| IS_AFFECTED=$(echo "$AFFECTED_JSON" | jq -r '.["${{ matrix.test-name }}"] // false') | ||||||||||||||||
| echo "skip=$([ "$IS_AFFECTED" = "true" ] && echo "false" || echo "true")" >> $GITHUB_OUTPUT | ||||||||||||||||
| if [ "$IS_AFFECTED" != "true" ]; then | ||||||||||||||||
| echo "Skipping ${{ matrix.test-name }} - not affected by changes" | ||||||||||||||||
| fi | ||||||||||||||||
|
|
||||||||||||||||
| - name: Checkout Repo | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||
| with: | ||||||||||||||||
| fetch-depth: 1 | ||||||||||||||||
|
|
@@ -338,73 +433,40 @@ jobs: | |||||||||||||||
| show-progress: false | ||||||||||||||||
|
|
||||||||||||||||
| - name: Setup | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| id: config | ||||||||||||||||
| uses: ./.github/actions/init-blacksmith | ||||||||||||||||
| with: | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| playwright-enabled: true | ||||||||||||||||
|
|
||||||||||||||||
| - name: Verify jq is installed | ||||||||||||||||
| shell: bash | ||||||||||||||||
| run: | | ||||||||||||||||
| if ! command -v jq &> /dev/null; then | ||||||||||||||||
| echo "jq not found, installing..." | ||||||||||||||||
| sudo apt-get update && sudo apt-get install -y jq | ||||||||||||||||
| fi | ||||||||||||||||
| jq --version | ||||||||||||||||
|
|
||||||||||||||||
| - name: Task Status | ||||||||||||||||
| id: task-status | ||||||||||||||||
| env: | ||||||||||||||||
| E2E_APP_CLERK_JS_DIR: ${{runner.temp}} | ||||||||||||||||
| E2E_APP_CLERK_UI_DIR: ${{runner.temp}} | ||||||||||||||||
| E2E_CLERK_JS_VERSION: "latest" | ||||||||||||||||
| E2E_CLERK_UI_VERSION: "latest" | ||||||||||||||||
| E2E_NEXTJS_VERSION: ${{ matrix.next-version }} | ||||||||||||||||
| E2E_PROJECT: ${{ matrix.test-project }} | ||||||||||||||||
| INTEGRATION_INSTANCE_KEYS: ${{ secrets.INTEGRATION_INSTANCE_KEYS }} | ||||||||||||||||
| run: | | ||||||||||||||||
| # Use turbo's built-in --affected flag to detect changes | ||||||||||||||||
| # This automatically uses GITHUB_BASE_REF in GitHub Actions | ||||||||||||||||
| TASK_COUNT=$(pnpm turbo run test:integration:${{ matrix.test-name }} --affected --dry=json 2>/dev/null | jq '.tasks | length' 2>/dev/null || echo "0") | ||||||||||||||||
|
|
||||||||||||||||
| if [ "$TASK_COUNT" -gt 0 ]; then | ||||||||||||||||
| AFFECTED=1 | ||||||||||||||||
| else | ||||||||||||||||
| AFFECTED=0 | ||||||||||||||||
| fi | ||||||||||||||||
|
|
||||||||||||||||
| echo "affected=${AFFECTED}" | ||||||||||||||||
| echo "affected=${AFFECTED}" >> $GITHUB_OUTPUT | ||||||||||||||||
|
|
||||||||||||||||
| - name: Version packages for snapshot | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| run: npm run version-packages:snapshot ci | ||||||||||||||||
|
|
||||||||||||||||
| - name: Verdaccio | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| uses: ./.github/actions/verdaccio | ||||||||||||||||
| with: | ||||||||||||||||
| publish-cmd: | | ||||||||||||||||
| if [ "$(pnpm config get registry)" = "https://registry.npmjs.org/" ]; then echo 'Error: Using default registry' && exit 1; else pnpm turbo build $TURBO_ARGS --only && pnpm changeset publish --no-git-tag --tag latest; fi | ||||||||||||||||
|
|
||||||||||||||||
| - name: Edit .npmrc [link-workspace-packages=false] | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| run: sed -i -E 's/link-workspace-packages=(deep|true)/link-workspace-packages=false/' .npmrc | ||||||||||||||||
|
|
||||||||||||||||
| - name: Install @clerk/backend in /integration | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| working-directory: ./integration | ||||||||||||||||
| run: | | ||||||||||||||||
| pnpm init | ||||||||||||||||
| pnpm config set minimum-release-age-exclude @clerk/* | ||||||||||||||||
| pnpm add @clerk/backend | ||||||||||||||||
|
|
||||||||||||||||
| # Install published packages from Verdaccio to test against real npm install scenarios | ||||||||||||||||
| # rather than local monorepo builds. Validates package structure, dependencies, and entry points. | ||||||||||||||||
| - name: Install @clerk/clerk-js in os temp | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| working-directory: ${{runner.temp}} | ||||||||||||||||
| run: | | ||||||||||||||||
| mkdir clerk-js && cd clerk-js | ||||||||||||||||
|
|
@@ -413,7 +475,7 @@ jobs: | |||||||||||||||
| pnpm add @clerk/clerk-js | ||||||||||||||||
|
|
||||||||||||||||
| - name: Install @clerk/ui in os temp | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| working-directory: ${{runner.temp}} | ||||||||||||||||
| run: | | ||||||||||||||||
| mkdir clerk-ui && cd clerk-ui | ||||||||||||||||
|
|
@@ -422,11 +484,11 @@ jobs: | |||||||||||||||
| pnpm add @clerk/ui | ||||||||||||||||
|
|
||||||||||||||||
| - name: Copy components @clerk/astro | ||||||||||||||||
| if: ${{ matrix.test-name == 'astro' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' && matrix.test-name == 'astro' }} | ||||||||||||||||
| run: cd packages/astro && pnpm copy:components | ||||||||||||||||
|
|
||||||||||||||||
| - name: Write all ENV certificates to files in integration/certs | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| uses: actions/github-script@v7 | ||||||||||||||||
| env: | ||||||||||||||||
| INTEGRATION_CERTS: "${{secrets.INTEGRATION_CERTS}}" | ||||||||||||||||
|
|
@@ -444,12 +506,12 @@ jobs: | |||||||||||||||
| } | ||||||||||||||||
|
|
||||||||||||||||
| - name: LS certs | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| working-directory: ./integration/certs | ||||||||||||||||
| run: ls -la && pwd | ||||||||||||||||
|
|
||||||||||||||||
| - name: Run Integration Tests | ||||||||||||||||
| if: ${{ steps.task-status.outputs.affected == '1' }} | ||||||||||||||||
| if: ${{ steps.affected.outputs.skip != 'true' }} | ||||||||||||||||
| id: integration-tests | ||||||||||||||||
| timeout-minutes: 25 | ||||||||||||||||
| run: pnpm turbo test:integration:${{ matrix.test-name }} $TURBO_ARGS | ||||||||||||||||
|
|
@@ -467,7 +529,7 @@ jobs: | |||||||||||||||
| VERCEL_AUTOMATION_BYPASS_SECRET: ${{ secrets.VERCEL_AUTOMATION_BYPASS_SECRET }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Upload test-results | ||||||||||||||||
| if: ${{ cancelled() || failure() }} | ||||||||||||||||
| if: ${{ !steps.affected.outputs.skip && (cancelled() || failure()) }} | ||||||||||||||||
| uses: actions/upload-artifact@v4 | ||||||||||||||||
|
Comment on lines
554
to
556
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Artifact upload guard is always false.
🔧 Suggested fix- if: ${{ !steps.affected.outputs.skip && (cancelled() || failure()) }}
+ if: ${{ steps.affected.outputs.skip != 'true' && (cancelled() || failure()) }}📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||
| with: | ||||||||||||||||
| name: playwright-traces-${{ github.run_id }}-${{ github.run_attempt }}-${{ matrix.test-name }}${{ matrix.next-version && format('-next{0}', matrix.next-version) || '' }} | ||||||||||||||||
|
|
@@ -477,7 +539,7 @@ jobs: | |||||||||||||||
| pkg-pr-new: | ||||||||||||||||
| name: Publish with pkg-pr-new | ||||||||||||||||
| if: ${{ github.event_name != 'pull_request' || github.event.pull_request.draft == false }} | ||||||||||||||||
| needs: [check-permissions, build-packages] | ||||||||||||||||
| needs: [check-permissions] | ||||||||||||||||
| runs-on: "blacksmith-8vcpu-ubuntu-2204" | ||||||||||||||||
| defaults: | ||||||||||||||||
| run: | ||||||||||||||||
|
|
@@ -500,10 +562,10 @@ jobs: | |||||||||||||||
| with: | ||||||||||||||||
| turbo-enabled: true | ||||||||||||||||
| node-version: 22 | ||||||||||||||||
| # turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| # turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| # turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| # turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
| turbo-signature: ${{ secrets.TURBO_REMOTE_CACHE_SIGNATURE_KEY }} | ||||||||||||||||
| turbo-summarize: ${{ env.TURBO_SUMMARIZE }} | ||||||||||||||||
| turbo-team: ${{ vars.TURBO_TEAM }} | ||||||||||||||||
| turbo-token: ${{ secrets.TURBO_TOKEN }} | ||||||||||||||||
|
|
||||||||||||||||
| - name: Publish with pkg-pr-new | ||||||||||||||||
| run: pnpm run build && pnpx pkg-pr-new@${{ vars.PKG_PR_NEW_VERSION || '0.0.49' }} publish --compact --pnpm './packages/*' | ||||||||||||||||
| run: pnpm turbo build $TURBO_ARGS && pnpx pkg-pr-new@${{ vars.PKG_PR_NEW_VERSION || '0.0.49' }} publish --compact --pnpm './packages/*' | ||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: clerk/javascript
Length of output: 709
🏁 Script executed:
Repository: clerk/javascript
Length of output: 1467
🏁 Script executed:
# Check the init-blacksmith action for more context cat -n .github/actions/init-blacksmith/action.ymlRepository: clerk/javascript
Length of output: 5321
Configure actionlint to recognize the Blacksmith runner label.
The
blacksmith-8vcpu-ubuntu-2204label is used in this workflow (lines 279, 404, and others) but lacks actionlint configuration to recognize it. Add an.github/actionlint.yamlto declare custom runner labels, or includeself-hostedin theruns-onvalue to satisfy linting without configuration.🧰 Tools
🪛 actionlint (1.7.11)
[error] 279-279: label "blacksmith-8vcpu-ubuntu-2204" is unknown. available labels are "windows-latest", "windows-latest-8-cores", "windows-2025", "windows-2025-vs2026", "windows-2022", "windows-11-arm", "ubuntu-slim", "ubuntu-latest", "ubuntu-latest-4-cores", "ubuntu-latest-8-cores", "ubuntu-latest-16-cores", "ubuntu-24.04", "ubuntu-24.04-arm", "ubuntu-22.04", "ubuntu-22.04-arm", "macos-latest", "macos-latest-xlarge", "macos-latest-large", "macos-26-xlarge", "macos-26-large", "macos-26", "macos-15-intel", "macos-15-xlarge", "macos-15-large", "macos-15", "macos-14-xlarge", "macos-14-large", "macos-14", "self-hosted", "x64", "arm", "arm64", "linux", "macos", "windows". if it is a custom label for self-hosted runner, set list of labels in actionlint.yaml config file
(runner-label)
🤖 Prompt for AI Agents