Skip to content

fix(ci): remediate reactlog package checks on main#114

Open
posit-shiny-automation[bot] wants to merge 2 commits into
mainfrom
shinycoreci/claude-remediation-25642867551-1-R-CMD-check.yaml
Open

fix(ci): remediate reactlog package checks on main#114
posit-shiny-automation[bot] wants to merge 2 commits into
mainfrom
shinycoreci/claude-remediation-25642867551-1-R-CMD-check.yaml

Conversation

@posit-shiny-automation
Copy link
Copy Markdown

Summary

Automated remediation for the failed package checks dispatched by rstudio/shinycoreci.

Failure context

Field Value
Target ref main
Remote workflow R-CMD-check.yaml
Remote conclusion failure
Remote run https://github.com/rstudio/reactlog/actions/runs/25642876486
Scheduler run https://github.com/rstudio/shinycoreci/actions/runs/25642867551

Closes #113

Automated remediation

Refs: #113
Remote workflow: R-CMD-check.yaml
Remote conclusion: failure
Remote run: https://github.com/rstudio/reactlog/actions/runs/25642876486
Scheduler run: https://github.com/rstudio/shinycoreci/actions/runs/25642867551
The man/reactlog_module.Rd file still contained the old example code
using `!require("shiny")` with a bare `return()` call. When shiny is
unavailable (check-depends-only job), the return() at top level causes
`Error: no function to return from, jumping to top level`.

Updated the Rd file to match the already-fixed R/shinyModule.R example,
wrapping the entire example body in `if (requireNamespace("shiny", quietly = TRUE))`.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@posit-shiny-automation
Copy link
Copy Markdown
Author

Verification still failing after 2 attempt(s) (final conclusion: failure). Manual review required.

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.

Automated fix: reactlog package checks failing on main

1 participant