Skip to content

Commit

Permalink
add alignment check to ci workflow
Browse files Browse the repository at this point in the history
we're adding the check in the android_tests as we already need an apk to run the tests, so we're not having to build multiple apks and slowing everything down

we considered after PR merge but that meant the person would only find out after that something they are using is not aligned
  • Loading branch information
mikescamell committed Feb 13, 2025
1 parent a45d8a0 commit 3eae73b
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,19 @@ jobs:
- name: Build
run: ./gradlew androidTestsBuild

- name: Find test APK path
id: find-apk
run: |
TEST_APK_PATH=$(find . -path '*/build/outputs/apk/play/debug/*.apk' -type f -print -quit)
echo "Found test APK at: $TEST_APK_PATH"
echo "apk_path=$TEST_APK_PATH" >> "$GITHUB_OUTPUT"
- name: Make script executable
run: chmod +x scripts/check_elf_alignment.sh

- name: Check native libraries alignment
run: ./scripts/check_elf_alignment.sh ${{ steps.find-apk.outputs.apk_path }}

- name: Run Android Tests
run: ./gradlew runFlankAndroidTests

Expand Down

0 comments on commit 3eae73b

Please sign in to comment.