Skip to content

release prep - #1

Merged
mstackhouse merged 7 commits into
mainfrom
dev
Apr 29, 2026
Merged

release prep#1
mstackhouse merged 7 commits into
mainfrom
dev

Conversation

@mstackhouse

@mstackhouse mstackhouse commented Apr 13, 2026

Copy link
Copy Markdown
Member

Summary

  • License: Changed from "All rights reserved" to Apache-2.0 for CRAN redistribution compliance
  • DESCRIPTION: Bumped version to 0.1.0, rewrote title/description, added SystemRequirements, cph role, moved logrx to Suggests
  • CRAN policy fixes: .onLoad no longer overwrites user options, setwd() protected with on.exit(), hardcoded internal IP removed, dplyr dependency eliminated, broken vignette URLs fixed
  • Documentation: Added missing @examples to 7 exported functions, fixed @return/@param gaps, added NEWS.md
  • CI/CD: Added GitHub Actions for R-CMD-check (6 platform matrix), pkgdown, R-hub, and test coverage
  • Build hygiene: Added rsconnect/, test_programs/, .github/ to .Rbuildignore
  • Test fixes: macOS temp dir portability, httr2 1.2.2 header compatibility

R CMD check --as-cran passes with 1 NOTE (new submission).

Test plan

  • R CMD check --as-cran passes locally (1 NOTE only: new submission)
  • All 584 tests pass (583 PASS + 1 SKIP on CRAN)
  • R-CMD-check GitHub Action passes on all 6 platforms
  • Verify on win-builder before CRAN submission
  • Review pkgdown site renders correctly

mstackhouse and others added 2 commits April 13, 2026 18:48
- Change license to Apache-2.0 for CRAN redistribution compliance
- Bump version to 0.1.0 for first release
- Rewrite DESCRIPTION title/description, add SystemRequirements and cph role
- Move logrx from Imports to Suggests (not used in R/ code)
- Fix .onLoad to not overwrite user-set options
- Change default abba.home.nfs.ip.address from hardcoded IP to NULL
- Protect setwd() calls with on.exit() in read_rproj.R and inst/logrx_workbench_submission.R
- Add file.exists() guard for /etc/rstudio/r-versions
- Handle NULL home_nfs_ip_address in configure_yaml.R
- Remove dplyr dependency from submit_job_slurm.R (use base R subsetting)
- Remove piped . usage in slurm_utils.R
- Add missing @examples to 7 exported functions
- Fix @return on create_batch, escape braces in Rd
- Document working_dir and ... params in abba_slurm_submit_job
- Replace installed.packages() with requireNamespace() in inst/ template
- Add rsconnect/, test_programs/, .claude/, CLAUDE.md to .Rbuildignore
- Fix broken Posit URLs in vignettes
- Fix _pkgdown.yml typo in function reference
- Fix test portability: tempdir() instead of /tmp/, httr2 header weak refs
- Add NEWS.md and CLAUDE.md

R CMD check --as-cran passes with 1 NOTE (new submission).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- R-CMD-check across Windows, macOS, and Ubuntu (release + devel)
- pkgdown site build and deployment to GitHub Pages
- R-hub multi-platform checks (manual dispatch)
- Test coverage with codecov integration
- Add .github to .Rbuildignore

Workflows match atorus-research/clinify configuration.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Apr 13, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

mstackhouse and others added 5 commits April 14, 2026 15:00
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove rsconnect/, plumber.R, notes.md from git tracking (contain
  internal server URLs, Azure Container Registry paths, IP addresses)
- Add them to .gitignore to prevent re-addition
- Replace real registry URL and username path in k8s_job_submission vignette
- Add *.Rcheck/ and *.tar.gz to .gitignore

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mstackhouse
mstackhouse merged commit c10b425 into main Apr 29, 2026
16 checks passed
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