-
-
Notifications
You must be signed in to change notification settings - Fork 4
E2e env actions #85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
E2e env actions #85
Changes from 1 commit
a75fa8b
ab67bec
22a7b88
610d928
6ebb0f3
f6e583d
e4322a4
4a81330
bc2f52f
27eed96
c9e4d1e
f5c4f53
860c7d4
4898cab
8b76ea8
f1008f7
9884d8a
682cafb
6a1ff90
38bd065
2f5e2a6
a3ea5d4
8f3fdcb
c7d9987
90d1bd2
bfd20d8
6d49bac
b2e0ef3
c35dbc6
9e75e06
dda3178
03cd24b
4b4e46f
02ff9a9
5695370
f399c35
4d87552
929ddaa
d9de05d
18421e8
93cf96a
db78229
37051d0
dade476
8529256
3b49d84
a4ca242
b1867a9
860e85f
032380e
eea10a6
1366275
1a08796
367c013
d7a349b
4459e30
4ee8ff1
9e04cea
aa884d6
a4d8ac3
a3cd1cb
b748908
8553525
cf5b15f
5827d99
ac3212f
f850b3e
a3886bc
af1ea7a
9c67713
87877ab
48f952e
7228375
d942920
c9620dc
6fe7aea
f8b417a
11952a4
cd57fed
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -187,7 +187,7 @@ runs: | |
|
|
||
| # Select Xcode version | ||
| - name: Select Xcode version | ||
| if: ${{ inputs.platform == 'ios' }} | ||
| if: ${{ inputs.platform == 'ios' && inputs.is-self-hosted == 'false' }} | ||
| run: sudo xcode-select -s /Applications/Xcode_${{ inputs.xcode-version }}.app | ||
| shell: bash | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bug: iOS Xcode Selection Fails on GitHub RunnersThe |
||
|
|
||
|
|
@@ -203,7 +203,7 @@ runs: | |
|
|
||
| # Install CocoaPods w/ cached bundler environment | ||
| - name: Install CocoaPods via bundler | ||
| if: ${{ inputs.platform == 'ios'}} | ||
| if: ${{ inputs.platform == 'ios' && inputs.is-self-hosted == 'false' }} | ||
| run: bundle exec pod install --repo-update --verbose | ||
| working-directory: ios | ||
| shell: bash | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Redundant Xcode Version Selection in CI
The
setup-e2e-envaction contains duplicate Xcode version selection steps for iOS. The first step (line 144) uses an incomplete path and lacks theshell: bashdirective, while the second step (line 196) uses the correct path and includes the shell directive. The second step overrides the first, rendering the initial selection redundant.