Speed up PR CI (#33239)
* Speed up PR CI By skipping some desktop tests in pull requests and instead only running them in the Merge Queue * Fix comment
This commit is contained in:
@@ -206,6 +206,8 @@ jobs:
|
|||||||
needs: prepare_ed
|
needs: prepare_ed
|
||||||
name: "Desktop Windows"
|
name: "Desktop Windows"
|
||||||
uses: ./.github/workflows/build_desktop_windows.yaml
|
uses: ./.github/workflows/build_desktop_windows.yaml
|
||||||
|
# Skip Windows builds on PRs, as the Linux amd64 build is enough of a smoke test and includes the screenshot tests
|
||||||
|
if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests')
|
||||||
strategy:
|
strategy:
|
||||||
matrix:
|
matrix:
|
||||||
arch: [x64, ia32, arm64]
|
arch: [x64, ia32, arm64]
|
||||||
@@ -223,10 +225,13 @@ jobs:
|
|||||||
arch: [amd64, arm64]
|
arch: [amd64, arm64]
|
||||||
runAllTests:
|
runAllTests:
|
||||||
- ${{ github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests') }}
|
- ${{ github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests') }}
|
||||||
# We ship static sqlcipher builds, so delegate testing the system builds to the merge queue
|
|
||||||
exclude:
|
exclude:
|
||||||
|
# We ship static sqlcipher builds, so delegate testing the system builds to the merge queue
|
||||||
- runAllTests: false
|
- runAllTests: false
|
||||||
sqlcipher: system
|
sqlcipher: system
|
||||||
|
# Additionally skip arm64 system builds on PRs, as the amd64 test is enough for a smoke test and includes the screenshot tests
|
||||||
|
- runAllTests: false
|
||||||
|
arch: arm64
|
||||||
with:
|
with:
|
||||||
sqlcipher: ${{ matrix.sqlcipher }}
|
sqlcipher: ${{ matrix.sqlcipher }}
|
||||||
arch: ${{ matrix.arch }}
|
arch: ${{ matrix.arch }}
|
||||||
@@ -236,6 +241,9 @@ jobs:
|
|||||||
needs: prepare_ed
|
needs: prepare_ed
|
||||||
name: "Desktop macOS"
|
name: "Desktop macOS"
|
||||||
uses: ./.github/workflows/build_desktop_macos.yaml
|
uses: ./.github/workflows/build_desktop_macos.yaml
|
||||||
|
# Skip macOS builds on PRs, as the Linux amd64 build is enough of a smoke test and includes the screenshot tests
|
||||||
|
# and we have a very low limit of concurrent macos runners (5) across the Github org.
|
||||||
|
if: github.event_name != 'pull_request' || contains(github.event.pull_request.labels.*.name, 'X-Run-All-Tests')
|
||||||
with:
|
with:
|
||||||
blob_report: true
|
blob_report: true
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user