Skip to content

Make build timeouts configurable #228

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.

Sign up for GitHub

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

Merged
shahmishal merged 1 commit into from
Jan 20, 2026
Merged

Make build timeouts configurable #228

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 16 additions & 4 deletions .github/workflows/swift_package_test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,10 @@ on:
type: string
description: "macOS command to build and test the package"
default: "xcrun swift test"
macos_build_timeout:
type: number
description: "The default step timeout in minutes"
default: 60
ios_pre_build_command:
type: string
description: "macOS command to execute before building the Swift package for iOS"
Expand All @@ -131,6 +135,10 @@ on:
type: string
description: "macOS command to build the package for iOS"
default: "xcrun swift build --build-tests --sdk \"$(xcrun --sdk iphoneos --show-sdk-path)\" --triple arm64-apple-ios"
ios_build_timeout:
type: number
description: "The default step timeout in minutes"
default: 60
linux_build_command:
type: string
description: "Linux command to build and test the package"
Expand Down Expand Up @@ -166,6 +174,10 @@ on:
Note that Powershell does not automatically exit if a subcommand fails. The Invoke-Program utility is available to propagate non-zero exit codes.
It is strongly encouraged to run all command using `Invoke-Program` unless you want to continue on error eg. `Invoke-Program git apply patch.diff` instead of `git apply patch.diff`.
default: "Invoke-Program swift test"
windows_build_timeout:
type: number
description: "The default step timeout in minutes"
default: 60
macos_env_vars:
description: "Newline separated list of environment variables"
type: string
Expand Down Expand Up @@ -266,7 +278,7 @@ jobs:
run: ${{ inputs.macos_pre_build_command }}
- name: Build / Test
run: ${{ inputs.macos_build_command }} ${{ (contains(matrix.swift_version, 'nightly') && inputs.swift_nightly_flags) || inputs.swift_flags }}
timeout-minutes: 60
timeout-minutes: ${{ inputs.macos_build_timeout }}

ios-build:
name: iOS (Build Only, Xcode ${{ matrix.xcode_version }} - ${{ matrix.os_version }} - ${{ matrix.arch }})
Expand Down Expand Up @@ -304,7 +316,7 @@ jobs:
run: ${{ inputs.ios_pre_build_command }}
- name: Build
run: ${{ inputs.ios_build_command }} ${{ (contains(matrix.swift_version, 'nightly') && inputs.swift_nightly_flags) || inputs.swift_flags }}
timeout-minutes: 60
timeout-minutes: ${{ inputs.ios_build_timeout }}

linux-build:
name: Linux (${{ matrix.swift_version }} - ${{ matrix.os_version }} - ${{ matrix.arch }})
Expand Down Expand Up @@ -780,12 +792,12 @@ jobs:
'@ >> $env:TEMP\test-script\run.ps1
# Docker build
- name: Docker Build / Test
timeout-minutes: 60
timeout-minutes: ${{ inputs.windows_build_timeout }}
if: ${{ inputs.enable_windows_docker }}
run: |
docker run -v ${{ github.workspace }}:C:\source -v $env:TEMP\test-script:C:\test-script ${{ steps.pull_docker_image.outputs.image }} powershell.exe -NoLogo -File C:\test-script\run.ps1
# Docker-less build
- name: Build / Test
timeout-minutes: 60
timeout-minutes: ${{ inputs.windows_build_timeout }}
if: ${{ !inputs.enable_windows_docker }}
run: powershell.exe -NoLogo -File $env:TEMP\test-script\run.ps1; exit $LastExitCode