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

Add CI through GitHub Actions #50

Closed
wants to merge 3 commits into from
Closed

Conversation

florianm
Copy link
Collaborator

@florianm florianm commented Jul 3, 2020

This PR adds a GHA workflow with minimal necessary additions to the package which tests vortexR on Windows Server 2019 and MacOS. There are other available environments, which currently fail to install R packages and system dependencies correctly until I find the correct incantations.

Any additional environments we can get to pass GHA could be added to the installation documentation for users.

The results of GHA runs of my fork live at https://github.com/florianm/vortexR/actions. If merged, the results will live at https://github.com/carlopacioni/vortexR/actions, from where a README badge can be generated.

If this PR gets merged, the following tasks have to be run by a package maintainer:

  • Add GHA badge from https://github.com/carlopacioni/vortexR/actions to README
  • If a gh-pages branch is not built from the first passing GHA workflow, it's possible that @carlopacioni as the repo maintainer has to run tic::use_tic() in order to let tic set up the necessary secrets and branches at carlopacioni/vortexR.
  • Once a gh-pages branch is built, set repo homepage to "gh-pages" branch.
  • Once the repo homepage is active, add repo homepage to repo settings and to DESCRIPTION and release an update to CRAN.

This change is Reviewable

Florian Mayer added 2 commits July 3, 2020 12:15
* add tic.yml workflow and tic.R script
* enable known passing environments
* DESC: add Encoding UTF-8 (required by RoxygenNote)
* Rbuildignore tic files
* gitignore `/doc` folder - tic builds docs in gh-pages branch
Error: M"ake sure you have Java Development Kit installed and correctly registered in R. If in doubt, re-run "R CMD javareconf" as root."
@florianm florianm added the docs Improvements to documentation label Jul 3, 2020
@florianm florianm added this to the Best practice milestone Jul 3, 2020
@florianm florianm self-assigned this Jul 3, 2020
@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.919% when pulling affdd5f on florianm:gha into b363538 on carlopacioni:master.

@florianm
Copy link
Collaborator Author

florianm commented Jan 8, 2024

Closing in favour of #53

@florianm florianm closed this Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements to documentation enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants