Skip to content

Fix: Remove comments from CI dependency list for parser safety#9

Merged
pplupo merged 1 commit into
masterfrom
update-r-version-4.5.0
Jun 12, 2025
Merged

Fix: Remove comments from CI dependency list for parser safety#9
pplupo merged 1 commit into
masterfrom
update-r-version-4.5.0

Conversation

@pplupo
Copy link
Copy Markdown
Owner

@pplupo pplupo commented Jun 12, 2025

This commit further refines the packages list within the r-lib/actions/setup-r-dependencies step of the GitHub Actions workflow (.github/workflows/R-CMD-check.yml).

To prevent any possible misinterpretation by the pak parser, all comments have been removed from this specific multiline block. The list now contains only the package names (packrat, rcmdcheck), each on its own line.

This is a stricter approach to avoid the persistent "Cannot parse packages" error.

This commit further refines the `packages` list within the
`r-lib/actions/setup-r-dependencies` step of the GitHub Actions workflow
(`.github/workflows/R-CMD-check.yml`).

To prevent any possible misinterpretation by the `pak` parser, all
comments have been removed from this specific multiline block. The list
now contains only the package names (`packrat`, `rcmdcheck`), each on
its own line.

This is a stricter approach to avoid the persistent "Cannot parse packages"
error.
@pplupo pplupo merged commit b72a314 into master Jun 12, 2025
1 check failed
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.

1 participant