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.

Sign up for GitHub

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

Jump to bottom

Change CI to docker based setup #191

Draft
wants to merge 13 commits into
base: master
Choose a base branch
from
Draft

Change CI to docker based setup #191

wants to merge 13 commits into from

Conversation

rstub
Copy link
Member

@rstub rstub commented Feb 21, 2019

Inspired by Rcpp's CI set-up. A Makefile is used to allow local testing in a controlled environment.

rstub added 10 commits February 19, 2019 15:25
@rstub
Change CI to docker based setup
f3f7aed
@rstub
Run tests with R CMD check
4e29397
@rstub
Skip tests on Appveyour due to missing TeX system
9fe50e7
@rstub
Remove depricated testthat::not()
f5858b0
@rstub
Add quiet option to compare to supress some warnings
f7b9c7b
@rstub
Revert "Run tests with R CMD check" and
aa3d55e 
       "Skip tests on Appveyour due to missing TeX system"

This reverts commits 4e29397 and
9fe50e7.
@rstub
Use testthat::test_dir() instead of devtools::test()
ee1c4a3 
Since devtools::test() is meant for interactive use, it does not fail if
any test fails. A good alternative is provided by testthat::test_dir().
That does require the package to be installed, though. To ease the
installation, /usr/local/lib/R/site-library in the docker image is made
world-writeable.
@rstub
Update some graphs for current tool chain
4fc78a2
@rstub
Define HOME inside the docker container
7ff8c26
@rstub
Make CODECOV_TOKEN available inside the docker container
ebd0f4c
@rstub rstub requested a review from bellackn February 21, 2019 09:41
@rstub
Copy link
Member Author

rstub commented Feb 21, 2019

This also addresses #189.

bellackn
bellackn approved these changes Sep 25, 2019
View reviewed changes
Copy link

@bellackn bellackn left a comment

Choose a reason for hiding this comment

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

Looks fine to me! :)

@rstub rstub marked this pull request as draft November 28, 2023 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bellackn bellackn bellackn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rstub @bellackn