Skip to content

Commit

Permalink
Require canary only for smithy-lang repo
Browse files Browse the repository at this point in the history
  • Loading branch information
ysaito1001 committed Mar 28, 2024
1 parent 9d12ead commit 6274ec1
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,11 @@ name: Test
on:
workflow_call:
inputs:
run_canary:
description: Whether to run the canary or not.
required: false
default: true
type: boolean
run_sdk_examples:
description: Whether to run the SDK example checks or not.
required: false
Expand Down Expand Up @@ -354,9 +359,7 @@ jobs:
- test-exotic-platform-support
- canary
# Run this job even if its dependency jobs fail
if: |
always() &&
${{ github.event.pull_request.head.repo.full_name == 'smithy-lang/smithy-rs' }}
if: always() && ${{ inputs.run_canary }}
runs-on: ubuntu-latest
name: Matrix Success
steps:
Expand All @@ -369,19 +372,16 @@ jobs:
# Pseudo-job that depends on matrix jobs so that we don't have to enter
# the myriad of test matrix combinations into GitHub's protected branch rules
require-all-but-canary:
# Should NOT depend on check-sdk-examples since that's an optional check
needs:
- generate
- test-codegen
- test-sdk
- test-rust-windows
- test-exotic-platform-support
# Run this job even if its dependency jobs fail
if: |
always() &&
${{ github.event.pull_request.head.repo.full_name != 'smithy-lang/smithy-rs' }}
if: always() && ${{ ! inputs.run_canary }}
runs-on: ubuntu-latest
name: Matrix Success
name: Matrix Success (from fork)
steps:
- name: Verify jobs succeeded
# Pinned to commit hash of v1.2.2
Expand Down

0 comments on commit 6274ec1

Please sign in to comment.