Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 19 additions & 14 deletions .github/workflows/checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,28 +11,33 @@ jobs:
name: Build and test
runs-on: ubuntu-latest
if: ${{ !github.event.pull_request.draft }}
env:
GRADLE_OPTS: "-Dorg.gradle.jvmargs=-Xmx3g -XX:+UseParallelGC"
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v6
- name: Set up JDK 17
uses: actions/setup-java@v4
uses: actions/setup-java@v5
with:
java-version: '17'
distribution: 'adopt'
- name: Build
run: ./gradlew clean build -x test -x ktlintMainSourceSetCheck
- name: Test
run: ./gradlew test --continue
- name: Setup Gradle
uses: gradle/actions/setup-gradle@v5
with:
# Read cache in PRs, write to cache after merge to main or develop
cache-read-only: ${{ github.ref != 'refs/heads/main' && github.ref != 'refs/heads/develop' }}
- name: Build and Test
run: ./gradlew assembleDebug testDebugUnitTest lintDebug

lint-kt:
name: Lint Kotlin
runs-on: macos-latest
if: ${{ !github.event.pull_request.draft }}
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v6
- name: Set up JDK 17
uses: actions/setup-java@v4
uses: actions/setup-java@v5
with:
java-version: '17'
distribution: 'adopt'
Expand All @@ -47,14 +52,14 @@ jobs:
scripts: ${{ 'readium/navigator/src/main/assets/_scripts' }}
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v6
- name: Install pnpm
uses: pnpm/action-setup@v4
with:
package_json_file: readium/navigator/src/main/assets/_scripts/package.json
run_install: false
- name: Setup cache
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 20
cache: 'pnpm'
Expand All @@ -67,8 +72,8 @@ jobs:
run: pnpm --dir "$scripts" run checkformat
- name: Check if bundled scripts are up-to-date
run: |
make scripts-legacy
git diff --exit-code --name-only src/main/assets/readium/scripts/*.js
make scripts-legacy
git diff --exit-code --name-only src/main/assets/readium/scripts/*.js
lint-new-ts:
name: Lint TypeScript
runs-on: macos-latest
Expand All @@ -77,14 +82,14 @@ jobs:
scripts: ${{ 'readium/navigators/web/internals/scripts' }}
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v6
- name: Install pnpm
uses: pnpm/action-setup@v4
with:
package_json_file: readium/navigators/web/internals/scripts/package.json
run_install: false
- name: Setup cache
uses: actions/setup-node@v4
uses: actions/setup-node@v6
with:
node-version: 20
cache: 'pnpm'
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ org.gradle.jvmargs=-Xmx2048m -Dfile.encoding=UTF-8
# When configured, Gradle will run in incubating parallel mode.
# This option should only be used with decoupled projects. More details, visit
# http://www.gradle.org/docs/current/userguide/multi_project_builds.html#sec:decoupled_projects
# org.gradle.parallel=true
org.gradle.parallel=true
# AndroidX package structure to make it clearer which packages are bundled with the
# Android operating system, and which are packaged with your app"s APK
# https://developer.android.com/topic/libraries/support-library/androidx-rn
Expand Down