Skip to content

Conversation

@johanna0321
Copy link
Contributor

Description of your changes

Change romic import from CRAN to github

Issue ticket number and link

#44

Type of change

  • Bug fix
  • New feature
    • Backwards Incompatible?
  • Refactoring / code clean-up
  • Documentation add / update
  • Automated Test
  • Other (please specify)

(If applicable) How has this been tested?

Testing as manual workflow on branch

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses issue #44 by modifying the R-CMD-check workflow to install the romic package from GitHub instead of CRAN. The change includes adding github::calico/romic to the extra-packages list and removing the r-devel test configuration from the build matrix.

Key Changes:

  • Added GitHub-sourced romic dependency to R-CMD-check workflow
  • Removed r-devel Ubuntu test configuration from the build matrix

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@PMSeitzer PMSeitzer left a comment

Choose a reason for hiding this comment

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

LGTM

@johanna0321 johanna0321 merged commit 9f64924 into main Dec 5, 2025
9 of 10 checks passed
@johanna0321 johanna0321 deleted the issue-44 branch December 5, 2025 19:05
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.

3 participants