Skip to content

Commit 0d766cd

Browse files
steve-the-edwardssteve-the-edwards
authored
Merge pull request #811 from square/ios-runners
Updated Github Actions Workflows
2 parents 73edd7e + e5821fb commit 0d766cd

File tree

5 files changed

+85
-28
lines changed

5 files changed

+85
-28
lines changed

‎.github/workflows/kotlin.yml

Lines changed: 72 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,12 @@ jobs :
2525
runs-on : ubuntu-latest
2626
timeout-minutes : 20
2727
steps :
28-
- uses : actions/checkout@v2
29-
- name : set up JDK 11.0.7
30-
uses : actions/setup-java@v1
28+
- uses : actions/checkout@v3
29+
- name : set up JDK 11
30+
uses : actions/setup-java@v3
3131
with :
32-
java-version : 11.0.7
32+
distribution: 'zulu'
33+
java-version : 11
3334

3435
## Actual task
3536
- uses : burrunan/gradle-cache-action@v1
@@ -64,12 +65,13 @@ jobs :
6465
runs-on : ubuntu-latest
6566
timeout-minutes : 20
6667
steps :
67-
- uses : actions/checkout@v2
68+
- uses : actions/checkout@v3
6869
- uses : gradle/wrapper-validation-action@v1
69-
- name : set up JDK 11.0.7
70-
uses : actions/setup-java@v1
70+
- name : set up JDK 11
71+
uses : actions/setup-java@v3
7172
with :
72-
java-version : 11.0.7
73+
distribution: 'zulu'
74+
java-version : 11
7375

7476
## Actual task
7577
- uses : burrunan/gradle-cache-action@v1
@@ -83,6 +85,57 @@ jobs :
8385
gradle-build-scan-report : false
8486
gradle-distribution-sha-256-sum-warning : false
8587

88+
# Report as Github Pull Request Check.
89+
- name : Publish Test Report
90+
uses : mikepenz/action-junit-report@v3
91+
if : always() # always run even if the previous step fails
92+
with :
93+
report_paths : '**/build/test-results/test/TEST-*.xml'
94+
95+
ios-tests :
96+
name : iOS Tests
97+
runs-on : macos-latest
98+
timeout-minutes : 20
99+
steps :
100+
- uses : actions/checkout@v3
101+
- uses : gradle/wrapper-validation-action@v1
102+
- name : set up JDK 11
103+
uses : actions/setup-java@v3
104+
with :
105+
distribution: 'zulu'
106+
java-version : 11
107+
108+
## iOS Specific Tests (for KMP ios actuals in core and runtime).
109+
- uses : burrunan/gradle-cache-action@v1
110+
name : Check with Gradle
111+
with :
112+
gradle-dependencies-cache-key : |
113+
gradle/libs.versions.toml
114+
arguments : |
115+
iosX64Test --stacktrace
116+
concurrent : true
117+
gradle-build-scan-report : false
118+
gradle-distribution-sha-256-sum-warning : false
119+
120+
## iOS Specific Tests w/ strict memory model (for KMP ios actuals in core and runtime).
121+
- uses : burrunan/gradle-cache-action@v1
122+
name : Check with Gradle
123+
with :
124+
gradle-dependencies-cache-key : |
125+
gradle/libs.versions.toml
126+
arguments : |
127+
iosX64Test -Pkotlin.native.binary.memoryModel=strict --stacktrace
128+
concurrent : true
129+
gradle-build-scan-report : false
130+
gradle-distribution-sha-256-sum-warning : false
131+
132+
# Report as Github Pull Request Check.
133+
- name : Publish Test Report
134+
uses : mikepenz/action-junit-report@v3
135+
if : always() # always run even if the previous step fails
136+
with :
137+
report_paths : '**/build/test-results/test/TEST-*.xml'
138+
86139
instrumentation-tests :
87140
name : Instrumentation tests
88141
runs-on : macos-latest
@@ -96,11 +149,12 @@ jobs :
96149
# Unclear that older versions actually honor command to disable animation.
97150
# Newer versions are reputed to be too slow: https://github.com/ReactiveCircus/android-emulator-runner/issues/222
98151
steps :
99-
- uses : actions/checkout@v2
100-
- name : set up JDK 11.0.7
101-
uses : actions/setup-java@v1
152+
- uses : actions/checkout@v3
153+
- name : set up JDK 11
154+
uses : actions/setup-java@v3
102155
with :
103-
java-version : 11.0.7
156+
distribution: 'zulu'
157+
java-version : 11
104158

105159
## Build before running tests, using cache.
106160
- uses : burrunan/gradle-cache-action@v1
@@ -127,7 +181,7 @@ jobs :
127181

128182
- name : Upload results
129183
if : ${{ always() }}
130-
uses : actions/upload-artifact@v2
184+
uses : actions/upload-artifact@v3
131185
with :
132186
name : instrumentation-test-results-${{ matrix.api-level }}
133187
path : ./**/build/reports/androidTests/connected/**
@@ -137,13 +191,14 @@ jobs :
137191
runs-on : ubuntu-latest
138192
timeout-minutes : 20
139193
steps :
140-
- uses : actions/checkout@v2
194+
- uses : actions/checkout@v3
141195

142-
- name : set up JDK 11.0.7
196+
- name : set up JDK 11
143197
if : env.MOBILE_DEV_API_KEY != null
144-
uses : actions/setup-java@v1
198+
uses : actions/setup-java@v3
145199
with :
146-
java-version : 11.0.7
200+
distribution: 'zulu'
201+
java-version : 11
147202
env :
148203
MOBILE_DEV_API_KEY : ${{ secrets.MOBILE_DEV_API_KEY }}
149204

‎.github/workflows/publish-snapshot.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,13 @@ jobs:
1414
timeout-minutes : 25
1515

1616
steps :
17-
- uses: actions/checkout@v2
17+
- uses: actions/checkout@v3
1818
- uses: gradle/wrapper-validation-action@v1
19-
- name: set up JDK 11.0.7
20-
uses: actions/setup-java@v1
19+
- name: set up JDK 11
20+
uses: actions/setup-java@v3
2121
with:
22-
java-version: 11.0.7
22+
distribution: 'zulu'
23+
java-version: 11
2324

2425
- name : Publish Snapshots
2526
run: |

‎.github/workflows/tutorials.yml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,12 @@ jobs:
1515
timeout-minutes: 15
1616
steps:
1717
# These setup steps should be common across all jobs in this workflow.
18-
- uses: actions/checkout@v2
19-
- name: set up JDK 11.0.7
20-
uses: actions/setup-java@v1
18+
- uses: actions/checkout@v3
19+
- name: set up JDK 11
20+
uses: actions/setup-java@v3
2121
with:
22-
java-version: 11.0.7
22+
distribution: 'zulu'
23+
java-version: 11
2324
- name: Build
2425
working-directory: ./samples/tutorial
2526
run: ./gradlew build --stacktrace

‎.github/workflows/validate-codeowners.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ jobs:
1111
name: Validate Codeowners
1212
runs-on: ubuntu-latest
1313
steps:
14-
- uses: actions/checkout@v2
14+
- uses: actions/checkout@v3
1515
# https://github.com/marketplace/actions/github-codeowners-validator
16-
- uses: mszostok/codeowners-validator@v0.6.0
16+
- uses: mszostok/codeowners-validator@v0.7.4
1717
with:
1818
# TODO(https://github.com/square/workflow-kotlin/issues/316) Add the owners check
1919
# back once this is fixed, or we implement a workaround.

‎.github/workflows/validate-documentation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
name: Lint Markdown files
1515
runs-on: ubuntu-latest
1616
steps:
17-
- uses: actions/checkout@v2
17+
- uses: actions/checkout@v3
1818
- name: Set up Ruby 2.6
1919
uses: actions/setup-ruby@v1
2020
with:

0 commit comments

Comments
 (0)