Skip to content

Commit

Permalink
fix cleanup params; test SSH on GHA
Browse files Browse the repository at this point in the history
this will include the .Rrofile option during 'make test', but not the
package checking -> so we test +/- multicore scheduler set
  • Loading branch information
mschubert committed Dec 16, 2023
1 parent dcdf014 commit da03092
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/check-standard.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ jobs:
run: |
R CMD INSTALL .
echo '.libPaths("~/work/_temp/Library")' >> ~/.Rprofile # cmq package in R
echo 'options(clustermq.scheduler="multicore")' >> ~/.Rprofile
echo "$(pwd)/tests/bin" >> $GITHUB_PATH # local cmq
sed -i "1iexport PATH=$(pwd)/tests/bin:\$PATH" ~/.bashrc || true # ssh cmq
Expand Down
2 changes: 1 addition & 1 deletion R/qsys_ssh.r
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ SSH = R6::R6Class("SSH",
private$workers_total = args$n_jobs
},

cleanup = function(quiet=FALSE) {
cleanup = function(success, timeout) {
private$finalize()
TRUE
}
Expand Down

0 comments on commit da03092

Please sign in to comment.