Skip to content

Commit 85645e6

Browse files
author
alex-omophub
committed
Reduce timeout for integration tests and optimize dependency installation
- Decreased the timeout for integration tests from 15 to 10 minutes to improve efficiency. - Updated the dependency installation step to only include necessary runtime and test dependencies, reducing installation time and resource usage.
1 parent 2038c12 commit 85645e6

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/integration-tests.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
# Only run on push to main/develop, not on PRs (saves ~7-8 min per PR)
1313
if: github.event_name == 'push' || github.event_name == 'workflow_dispatch'
1414
runs-on: ubuntu-latest
15-
timeout-minutes: 15
15+
timeout-minutes: 10
1616
env:
1717
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}
1818
OMOPHUB_API_KEY: ${{ secrets.OMOPHUB_API_KEY }}
@@ -26,10 +26,15 @@ jobs:
2626
r-version: 'release'
2727
use-public-rspm: true
2828

29+
# Install only the runtime + test dependencies the integration tests
30+
# actually need. The default `needs: check` pulls in all Suggests
31+
# (rmarkdown, knitr, etc.) which require heavy system libraries
32+
# (pandoc, libxml2-dev, cmake, libharfbuzz-dev, ...) and take 10+
33+
# minutes to install. Integration tests only need httr2 + testthat.
2934
- uses: r-lib/actions/setup-r-dependencies@v2
3035
with:
3136
extra-packages: any::testthat, any::devtools
32-
needs: deploy
37+
dependencies: '"hard"'
3338
cache-version: 2
3439

3540
- name: Run integration tests

0 commit comments

Comments
 (0)