Skip to content

Commit 4bff09f

Browse files
committed
install plotly in a separate R session
1 parent 8c54d4d commit 4bff09f

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

inst/build-push-comment.R

+5-2
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,11 @@ if (tpr != "false" && tpr != "") {
5454
base_pngs <- dir(base_dir, pattern = "\\.png$")
5555
# Re-run (current) test suite with master branch if it's missing any tests
5656
if (!all(this_pngs %in% base_pngs)) {
57+
cat(base_hash, file = "base_hash.txt")
5758
dir.create(base_dir, showWarnings = FALSE)
58-
devtools::install_github("ropensci/plotly", ref = base_hash)
59+
system2("Rscript",
60+
c("-e", shQuote("devtools::install_github('ropensci/plotly', ref = readLines('base_hash.txt'))"))
61+
)
5962
print("Rerunning tests with master")
6063
try(source("../plotly/tests/testthat.R", chdir = TRUE))
6164
}
@@ -107,7 +110,7 @@ if (tpr != "false" && tpr != "") {
107110
# strip any leading/trailing whitespace in urls
108111
hashes$url <- sub("\\s$", "", sub("^\\s", "", hashes$url))
109112
hashes <- hashes[hashes$commit %in% c(this_hash, base_hash), ]
110-
devtools::install("../plotly")
113+
system("R CMD INSTALL ../plotly")
111114
diffs <- character()
112115
for (i in tests) {
113116
test_info <- hashes[hashes$test %in% i, ]

0 commit comments

Comments
 (0)