Skip to content

Add initial R-universe Github action - #79

Open
sarahkelley42 wants to merge 5 commits into
develfrom
r-universe-testing
Open

sarahkelley42 wants to merge 5 commits into
develfrom
r-universe-testing

Conversation

@sarahkelley42

@sarahkelley42 sarahkelley42 commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Testing changes to the yml, not ready for merge yet.

@sarahkelley42
sarahkelley42 marked this pull request as ready for review September 14, 2026 13:00
@sarahkelley42

Copy link
Copy Markdown
Contributor Author

@kuwisdelu I could use your input on this; I believe this is the recommended setup per the bioconductor docs, with a variation I threw in from the r-universe-org readme because I'm not totally sure what the org fields actually do in these jobs. I'm not sure yet whether this actually catches test failures or if it's strictly a build.

Do you happen to have an example code change that might break things that I can cherry-pick and test with? Hoping to sanity check before calling this merge-ready.

@sarahkelley42

Copy link
Copy Markdown
Contributor Author

Although, early positive signal, we do get some failure:

  ✖ ERROR: At least 80% of man pages documenting exported objects must have
  runnable examples.
    The following pages do not:
      • Cardinal-package.Rd
      • ...
      • writeMSIData.Rd

https://github.com/Cardinalverse/Cardinal/actions/runs/34848819264/job/103994206520?pr=79

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