Skip to content

Commit f45814d

Browse files
committed
tweaks to CI steps
1 parent d735f24 commit f45814d

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

.github/workflows/ci.yaml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
shell: bash
3535
run: |
3636
./gradlew jvmApiCheck
37-
./gradlew -Ptest.java.version=${{ matrix.java-version }} -Paws.sdk.kotlin.crt.disableCrossCompile=true jvmTest --stacktrace
37+
./gradlew -Ptest.java.version=${{ matrix.java-version }} jvmTest --stacktrace
3838
- name: Save Test Reports
3939
if: failure()
4040
uses: actions/upload-artifact@v4
@@ -58,7 +58,7 @@ jobs:
5858
uses: ./.github/actions/setup-build
5959
- name: Build and Test ${{ env.PACKAGE_NAME }}
6060
run: |
61-
./gradlew -Paws.sdk.kotlin.crt.disableCrossCompile=true build
61+
./gradlew build
6262
- name: Save Test Reports
6363
if: failure()
6464
uses: actions/upload-artifact@v4
@@ -68,7 +68,6 @@ jobs:
6868

6969
# build and test for targets: jvm, linuxX64
7070
# cross compile for: linuxX64, linuxArm64
71-
# TODO - add mingw as cross compile target
7271
linux:
7372
runs-on: ubuntu-22.04
7473
steps:
@@ -97,7 +96,7 @@ jobs:
9796
if: failure()
9897
uses: actions/upload-artifact@v4
9998
with:
100-
name: test-reports-linux
99+
name: test-reports-linux-jvm
101100
path: '**/build/reports'
102101
retention-days: 15
103102

@@ -130,7 +129,7 @@ jobs:
130129
RUN_CONTAINER_TEST=./native-test-binaries/.github/scripts/run-container-test.py
131130
$RUN_CONTAINER_TEST --distro ${{ matrix.distro }} --arch ${{ matrix.arch }} --test-bin-dir ./native-test-binaries/aws-crt-kotlin/build/bin
132131
133-
# windows JVM
132+
# windows JVM & native
134133
windows:
135134
runs-on: windows-2022
136135
steps:
@@ -141,7 +140,7 @@ jobs:
141140
uses: ./.github/actions/setup-build
142141
- name: Build and Test ${{ env.PACKAGE_NAME }}
143142
run: |
144-
./gradlew -P"aws.sdk.kotlin.crt.disableCrossCompile"=true build
143+
./gradlew build
145144
- name: Save Test Reports
146145
if: failure()
147146
uses: actions/upload-artifact@v4
@@ -150,5 +149,3 @@ jobs:
150149
path: '**/build/reports'
151150

152151
# TODO - native test reports?
153-
# TODO - windows native
154-

0 commit comments

Comments
 (0)