Update CI workflows for self-hosted runners#1627
Merged
gemcoder21 merged 20 commits intomainfrom Jan 24, 2026
Merged
Conversation
Changed the 'runs-on' property to use self-hosted macOS runners in both CI and UI test workflows. Added conditional execution for caching and sccache steps to only run on GitHub-hosted environments. Removed unnecessary sccache stats verification from UI tests.
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
Eliminated the 'Select Xcode' step from both ci.yml and ui_tests.yml GitHub Actions workflows as it is no longer necessary.
Moves SCCACHE_GHA_ENABLED environment variable setup into a conditional step for GitHub-hosted runners in both ci.yml and ui_tests.yml. This ensures the variable is only set when appropriate, improving workflow flexibility.
Added a conditional to only install 'just' with Homebrew when the workflow is running on GitHub-hosted runners in both ci.yml and ui_tests.yml. This prevents unnecessary installation steps on self-hosted or other environments.
Added commented lines for 'runs-on: macos-26' in both ci.yml and ui_tests.yml to document the previous runner configuration while continuing to use self-hosted macOS runners.
The CI workflow now explicitly cleans DerivedData, builds the project, and runs tests without building. Added a new 'test-without-building' recipe to the justfile and updated the workflow to use it, improving build efficiency and clarity.
Consolidated test-all and test-without-building into a single _test action recipe to reduce duplication. Updated test-all and test-without-building to use the new _test action with appropriate arguments.
Replaces the 'build' step in CI with 'build-for-testing' and refactors the justfile to introduce reusable build and test recipes. Adds variables for derived data and build flags, and streamlines test invocation for specific targets. This improves maintainability and flexibility of build and test automation.
Introduces a new XCTest plan file for app unit tests and updates the Xcode project to include it in the resources build phase. This enables more flexible and configurable test execution for the GemTests target.
Removed unused 'xcbeautify' and 'list' recipes for clarity. Added '@' to several commands for consistent output suppression and improved the 'download-wallet-core' recipe to use silent curl.
Sets RUSTC_WRAPPER in the sccache setup step for GitHub-hosted runners instead of globally in ci.yml and ui_tests.yml. This ensures the environment variable is only set when appropriate.
Introduces a new 'test-ui-without-building' Justfile recipe and updates the UI test workflow to use it. This allows running UI tests without rebuilding, improving CI efficiency. Also adds a separate build-for-testing step to the workflow.
Replaces the commented-out 'rm -rf build/DerivedData' step with an active 'just clean' command in the CI workflow for improved consistency and maintainability.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed the 'runs-on' property to use self-hosted macOS runners in both CI and UI test workflows. Added conditional execution for caching and sccache steps to only run on GitHub-hosted environments. Removed unnecessary sccache stats verification from UI tests.