letro/letro-ios
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases 2 Wiki Activity

Only install imagemagick when needed.

Browse Source 
Workaround for issues installing python from brew on a runner right now.
This commit is contained in:
Doug
2024-03-11 11:45:19 +00:00
committed by Doug
parent 11ca202165
commit eebe5f362b
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/pr-build.yml vendored
View File

@@ -29,6 +29,8 @@ jobs:
${{ runner.os }}-gems- ${{ runner.os }}-gems-
- name: Setup environment - name: Setup environment
env:
CI_WORKFLOW: PR_BUILD
run: run:
source ci_scripts/ci_common.sh && setup_github_actions_environment source ci_scripts/ci_common.sh && setup_github_actions_environment

6
ci_scripts/ci_common.sh
View File

@@ -41,7 +41,11 @@ setup_github_actions_environment() {
unset HOMEBREW_NO_INSTALL_FROM_API unset HOMEBREW_NO_INSTALL_FROM_API
export HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK=1 export HOMEBREW_NO_INSTALLED_DEPENDENTS_CHECK=1
brew update && brew install xcodegen swiftformat git-lfs imagemagick a7ex/homebrew-formulae/xcresultparser brew update && brew install xcodegen swiftformat git-lfs a7ex/homebrew-formulae/xcresultparser
if [ "$CI_WORKFLOW" = "PR_BUILD" ]; then
brew install imagemagick
fi
# brew "swiftlint" # Fails on the CI: `Target /usr/local/bin/swiftlint Target /usr/local/bin/swiftlint already exists`. Installed through https://github.com/actions/virtual-environments/blob/main/images/macos/macos-12-Readme.md#linters # brew "swiftlint" # Fails on the CI: `Target /usr/local/bin/swiftlint Target /usr/local/bin/swiftlint already exists`. Installed through https://github.com/actions/virtual-environments/blob/main/images/macos/macos-12-Readme.md#linters