Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a CI system #11

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Krannich479
Copy link
Collaborator

I implemented a continuous integration (CI) system to check for a successful build of VariantSurvival under various operating systems and R versions. The CI system only verifies a successful resolution of dependencies and installation of the VariantSurvival R package, not its functionality (yet)!
Tested operating systems are:

  • MacOS (latest)
  • Ubuntu Linux (latest, 20.04 LTS)
  • Windows (latest)

Further, the CI workflow file aids the experienced user with installation instructions for OS dependent system requirements. At the moment the CI system is set in a way that it automatically checks PRs and commits to the main branch.

If you'd like me to add a (i.e. your) particular OS + R version for the CI system to check, let me know.

@Krannich479 Krannich479 added the enhancement General improvements of the software label Jan 5, 2023
@Krannich479 Krannich479 self-assigned this Jan 5, 2023
@msarrias
Copy link
Collaborator

msarrias commented Jan 8, 2023

I'm not sure how to review/test this, can we have a quick meeting and go through it together? @Krannich479 @HibaBenAribi

@Krannich479
Copy link
Collaborator Author

Krannich479 commented Jan 11, 2023

Absolutely, I'll be back at work and available next week (from Jan 16th on). In a nutshell, all I did was testing the installation process of VariantSurvival using various OS and R versions. You can see that VariantSurvival successfully runs on Linux, MasOS and Windows in the GH Actions report here. The CI workflow in my fork that runs these tests can be seen here, it just installs some OS dependent libraries and executes the commands mentioned in our README.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement General improvements of the software
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants