Check final status for workflow jobs by Goooler · Pull Request #23 · sozercan/kaset

Goooler · 2026-01-03T00:25:51Z

Also groups all required workflow jobs into the ci.yml file.

Benefits for

You can enable the check in the repo settings, this enables auto-merge for PRs.

Also groups all required workflow jobs into the ci.yml file. You can enable the check in repo settings, this unblocks auto-merge for PRs.

Copilot

Pull request overview

This PR consolidates all CI workflow jobs into a single ci.yml file and introduces a final-status check job that can be used as a required status check in branch protection rules. This simplification enables auto-merge functionality for pull requests.

Key changes:

Merges three separate workflow files (tests.yml, lint.yml, dev-build.yml) into one unified ci.yml
Adds a final-status job that aggregates the results of all required jobs for branch protection
Removes path-specific filtering to trigger CI on all pull requests and main branch pushes

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
.github/workflows/tests.yml	Removed - unit and UI test jobs migrated to ci.yml
.github/workflows/lint.yml	Removed - SwiftLint and SwiftFormat jobs migrated to ci.yml
.github/workflows/dev-build.yml	Removed - build and DMG creation job migrated to ci.yml
.github/workflows/ci.yml	New consolidated workflow containing all jobs plus final-status check

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Goooler · 2026-01-03T00:41:22Z

The unit test failures are unrelated, may fix them first and rebase this PR.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Goooler · 2026-01-03T03:15:38Z

It works fine, and the real diff is:

`diff dev-build.yml ci.yml`

1c1
< name: Dev Build
---
> name: CI
9,15c9,10
<     branches: [main]
<     paths:
<       - "**.swift"
<       - "**.xcodeproj/**"
<       - "**.xcworkspace/**"
<       - "Assets.xcassets/**"
<       - ".github/workflows/dev-build.yml"
---
>     branches:
>       - main
17,22d11
<     paths:
<       - "**.swift"
<       - "**.xcodeproj/**"
<       - "**.xcworkspace/**"
<       - "Assets.xcassets/**"
<       - ".github/workflows/dev-build.yml"
26c15,16
<   build:
---
>   swiftlint:
>     name: SwiftLint
28c18,24
<     timeout-minutes: 30
---
>     timeout-minutes: 10
>     steps:
>       - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
>       - run: brew install swiftlint
>       - name: Run SwiftLint
>         run: |
>           swiftlint lint --strict --reporter github-actions-logging
29a26,29
>   swiftformat:
>     name: SwiftFormat
>     runs-on: macos-26
>     timeout-minutes: 10
31,32c31,38
<       - name: Checkout code
<         uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
---
>       - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
>       - run: brew install swiftformat
>       - name: Check formatting
>         run: |
>           swiftformat --lint . 2>&1 || {
>             echo "::error::Code is not formatted. Run 'swiftformat .' locally and commit the changes."
>             exit 1
>           }
33a40,45
>   macos_unit_tests:
>     name: macOS Unit Tests
>     timeout-minutes: 20
>     runs-on: macos-26
>     steps:
>       - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
35a48,69
>       - name: Show Xcode version
>         run: xcodebuild -version
>       - name: Run unit tests
>         run: |
>           set -o pipefail
>           xcodebuild \
>             -project Kaset.xcodeproj \
>             -scheme Kaset \
>             -destination 'platform=macOS' \
>             -derivedDataPath ./build \
>             -only-testing:KasetTests \
>             CODE_SIGN_IDENTITY="" \
>             CODE_SIGNING_REQUIRED=NO \
>             CODE_SIGNING_ALLOWED=NO \
>             test
>       - name: Upload test results on failure
>         if: failure()
>         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v4
>         with:
>           name: macOS-TestResults
>           path: ./build/*.xcresult
>           retention-days: 7
36a71,78
>   macos_ui_tests:
>     name: macOS UI Tests
>     timeout-minutes: 30
>     runs-on: macos-26
>     steps:
>       - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
>       - name: Select Xcode version
>         run: sudo xcode-select -s /Applications/Xcode_26.2.app/Contents/Developer
38a81,100
>       - name: Run UI tests
>         run: |
>           set -o pipefail
>           xcodebuild \
>             -project Kaset.xcodeproj \
>             -scheme Kaset \
>             -destination 'platform=macOS' \
>             -derivedDataPath ./build \
>             -only-testing:KasetUITests \
>             CODE_SIGN_IDENTITY="" \
>             CODE_SIGNING_REQUIRED=NO \
>             CODE_SIGNING_ALLOWED=NO \
>             test
>       - name: Upload test results on failure
>         if: failure()
>         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v4
>         with:
>           name: macOS-UITestResults
>           path: ./build/*.xcresult
>           retention-days: 7
39a102,111
>   build:
>     name: Build
>     runs-on: macos-26
>     timeout-minutes: 30
>     steps:
>       - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
>       - name: Select Xcode version
>         run: sudo xcode-select -s /Applications/Xcode_26.2.app/Contents/Developer
>       - name: Show Xcode version
>         run: xcodebuild -version
43d114
< 
58d128
< 
90d159
< 
97a167,186
> 
>   # Status check that is required in branch protection rules.
>   final-status:
>     needs:
>       - swiftlint
>       - swiftformat
>       - macos_unit_tests
>       - macos_ui_tests
>       - build
>     runs-on: ubuntu-latest
>     if: always()
>     steps:
>       - name: Check
>         run: |
>           results=$(tr -d '\n' <<< '${{ toJSON(needs.*.result) }}')
>           if ! grep -q -v -E '(failure|cancelled)' <<< "$results"; then
>             echo "One or more required jobs failed"
>             exit 1
>           fi
>           echo "All required jobs completed successfully."

sozercan · 2026-01-07T19:25:22Z

+  pull_request:
+  workflow_dispatch:
+
+jobs:


i like this but its removing filtering based on files so a change in readme doesn't trigger a full ci run

can we use something like https://github.com/dorny/paths-filter to add a filter

It's intended. Removing all filters ensures all file changes will trigger the workflow.

sozercan · 2026-01-07T19:25:48Z

+    runs-on: macos-26
+    steps:
+      - uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v4
+      - uses: actions/cache@v5


please make sure to pin to sha for all actions

Reverted! Will land it in a separate PR.

f03c4be

This reverts commit 5111af8.

# Conflicts: # .github/workflows/dev-build.yml # .github/workflows/lint.yml # .github/workflows/tests.yml

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add a final-status check on CI

72b53ee

Also groups all required workflow jobs into the ci.yml file. You can enable the check in repo settings, this unblocks auto-merge for PRs.

Copilot AI review requested due to automatic review settings January 3, 2026 00:25

Copilot started reviewing on behalf of Goooler January 3, 2026 00:26 View session

Copilot AI reviewed Jan 3, 2026

View reviewed changes

Comment thread .github/workflows/ci.yml

Comment thread .github/workflows/ci.yml

Update .github/workflows/ci.yml

7416951

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Merge branch 'sozercan:main' into ci-checks

835fd78

Goooler requested a review from Copilot January 3, 2026 01:21

Copilot started reviewing on behalf of Goooler January 3, 2026 01:22 View session

Copilot AI reviewed Jan 3, 2026

View reviewed changes

Comment thread .github/workflows/ci.yml

Goooler changed the title ~~Add a final-status check on CI~~ Check final-status for workflow jobs Jan 3, 2026

Goooler changed the title ~~Check final-status for workflow jobs~~ Check final status for workflow jobs Jan 3, 2026

Goooler added 3 commits January 4, 2026 09:18

Merge remote-tracking branch 'origin/main' into ci-checks

cd74f3a

Rename jobs

93584c1

Cache build dirs

5111af8

sozercan reviewed Jan 7, 2026

View reviewed changes

Goooler added 2 commits January 8, 2026 07:56

Merge remote-tracking branch 'origin/main' into ci-checks

abc4454

Revert "Cache build dirs"

f03c4be

This reverts commit 5111af8.

jacksongoode mentioned this pull request Jan 26, 2026

About macOS version compatibility #9

Open

Goooler added 4 commits January 26, 2026 13:14

Merge remote-tracking branch 'origin/main' into ci-checks

427075e

# Conflicts: # .github/workflows/dev-build.yml # .github/workflows/lint.yml # .github/workflows/tests.yml

Fix merge

6b0b489

Update comments

70900ae

Merge remote-tracking branch 'origin/main' into ci-checks

509fc1d

# Conflicts: # .github/workflows/dev-build.yml # .github/workflows/lint.yml # .github/workflows/tests.yml

Goooler requested a review from Copilot February 28, 2026 05:59

Copilot started reviewing on behalf of Goooler February 28, 2026 06:00 View session

Copilot AI reviewed Feb 28, 2026

View reviewed changes

Fix merge

935ce1f

Goooler force-pushed the ci-checks branch from e579180 to 935ce1f Compare February 28, 2026 06:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Check final status for workflow jobs#23

Check final status for workflow jobs#23
Goooler wants to merge 13 commits into
sozercan:mainfrom
Goooler:ci-checks

Goooler commented Jan 3, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Goooler commented Jan 3, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Goooler commented Jan 3, 2026

Uh oh!

sozercan Jan 7, 2026

Uh oh!

Goooler Jan 7, 2026

Uh oh!

sozercan Jan 7, 2026

Uh oh!

Goooler Jan 7, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Goooler commented Jan 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Goooler commented Jan 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Goooler commented Jan 3, 2026

Uh oh!

sozercan Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

Goooler Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

sozercan Jan 7, 2026

Choose a reason for hiding this comment

Uh oh!

Goooler Jan 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Goooler commented Jan 3, 2026 •

edited

Loading

Goooler Jan 7, 2026 •

edited

Loading