Skip to content

Fix Quarto Docs CI: swap invalid param for RSPM binary repo - #345

Merged
shajoezhu merged 2 commits into
mainfrom
copilot/fix-quarto-docs-job
Aug 6, 2026
Merged

Fix Quarto Docs CI: swap invalid param for RSPM binary repo#345
shajoezhu merged 2 commits into
mainfrom
copilot/fix-quarto-docs-job

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

The "Quarto Docs 📕" job was failing because deps-installation-method: setup-r-dependencies is not a valid input for the pinned insightsengineering/setup-r-dependencies@47235b81 action version (silently ignored), leaving pak to compile Deriv from source — which fails due to missing internal R API symbols (R_ClosureFormals, Rf_allocLang) in the build environment.

Changes

  • .github/workflows/docs.yaml: Remove the unsupported deps-installation-method input; replace with repository-list pointing to the Posit Package Manager binary repo for Ubuntu 22.04 (jammy):
- name: Install dependencies ⛓️
  uses: insightsengineering/setup-r-dependencies@47235b81d6af3f63d38523dd1f8c09dc9150b76a
  with:
    repository-list: https://packagemanager.posit.co/cran/__linux__/jammy/latest

This directs pak to install pre-built binaries instead of compiling from source, bypassing the Deriv compilation failure.

Co-authored-by: shajoezhu <3692541+shajoezhu@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Quarto Docs Fix Quarto Docs CI: swap invalid param for RSPM binary repo Aug 6, 2026
Copilot AI requested a review from shajoezhu August 6, 2026 05:29

@shajoezhu shajoezhu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@shajoezhu
shajoezhu marked this pull request as ready for review August 6, 2026 05:31
@shajoezhu
shajoezhu enabled auto-merge (squash) August 6, 2026 05:31
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename            Stmts    Miss  Cover    Missing
----------------  -------  ------  -------  ---------
R/fda-table_02.R       25       1  96.00%   77
R/utils.R              40      40  0.00%    13-102
TOTAL                  65      41  36.92%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 4f90b11

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

28 tests    1 ✅  41s ⏱️
28 suites  27 💤
 1 files     0 ❌

Results for commit 4f90b11.

♻️ This comment has been updated with latest results.

@shajoezhu shajoezhu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@shajoezhu
shajoezhu merged commit 4f90b11 into main Aug 6, 2026
51 checks passed
@shajoezhu
shajoezhu deleted the copilot/fix-quarto-docs-job branch August 6, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants