Skip to content

Commit 6a7d635

Browse files
authored
feat: add homebrew_update parameter to toggle brew update (#131)
* refactor(ios): move brew update into separate step * feat(ios): add homebrew_update parameter to setup_macos_executor command * feat(android): add homebrew_update parameter to android_test job * feat(ios): add homebrew_update parameter to ios_build job * feat(ios): add homebrew_update parameter to ios_build_and_test job
1 parent 49c5e4a commit 6a7d635

File tree

4 files changed

+26
-1
lines changed

4 files changed

+26
-1
lines changed

src/commands/setup_macos_executor.yml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@ parameters:
99
description: Should we cache after brew install? Defaults to true
1010
type: boolean
1111
default: true
12+
homebrew_update:
13+
description: Should we run brew update? Defaults to true
14+
type: boolean
15+
default: true
1216

1317
steps:
1418
- run:
@@ -45,10 +49,16 @@ steps:
4549
name: Verify node version
4650
command: node --version
4751

52+
- when:
53+
condition: <<parameters.homebrew_update>>
54+
steps:
55+
- run:
56+
name: Update brew
57+
command: brew update >/dev/null
58+
4859
- run:
4960
name: Configure Detox Environment
5061
command: |
51-
brew update >/dev/null
5262
HOMEBREW_NO_AUTO_UPDATE=1 brew tap wix/brew >/dev/null
5363
HOMEBREW_NO_AUTO_UPDATE=1 brew tap homebrew/cask >/dev/null
5464
HOMEBREW_NO_AUTO_UPDATE=1 brew install applesimutils >/dev/null

src/jobs/android_test.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,13 +72,18 @@ parameters:
7272
description: Should we cache after brew install? Defaults to true
7373
type: boolean
7474
default: true
75+
homebrew_update:
76+
description: Should we run brew update? Defaults to true
77+
type: boolean
78+
default: true
7579

7680
steps:
7781
- attach_workspace:
7882
at: <<parameters.workspace_root>>
7983
- setup_macos_executor:
8084
node_version: <<parameters.node_version>>
8185
homebrew_cache: <<parameters.homebrew_cache>>
86+
homebrew_update: <<parameters.homebrew_update>>
8287
- yarn_install:
8388
cache: <<parameters.yarn_cache>>
8489
- when:

src/jobs/ios_build.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,10 @@ parameters:
6969
description: Should we cache after brew install? Defaults to true
7070
type: boolean
7171
default: true
72+
homebrew_update:
73+
description: Should we run brew update? Defaults to true
74+
type: boolean
75+
default: true
7276

7377
steps:
7478
- when:
@@ -83,6 +87,7 @@ steps:
8387
- setup_macos_executor:
8488
node_version: <<parameters.node_version>>
8589
homebrew_cache: <<parameters.homebrew_cache>>
90+
homebrew_update: <<parameters.homebrew_update>>
8691
- yarn_install:
8792
cache: <<parameters.yarn_cache>>
8893
cache_folder: <<parameters.yarn_cache_folder>>

src/jobs/ios_build_and_test.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,10 @@ parameters:
8787
description: Should we cache after brew install? Defaults to true
8888
type: boolean
8989
default: true
90+
homebrew_update:
91+
description: Should we run brew update? Defaults to true
92+
type: boolean
93+
default: true
9094

9195
steps:
9296
- when:
@@ -101,6 +105,7 @@ steps:
101105
- setup_macos_executor:
102106
node_version: <<parameters.node_version>>
103107
homebrew_cache: <<parameters.homebrew_cache>>
108+
homebrew_update: <<parameters.homebrew_update>>
104109
- ios_simulator_start:
105110
device: <<parameters.device>>
106111
- yarn_install:

0 commit comments

Comments
 (0)