Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
d9c7ee0
wip upgrade to plotly.js 2.0
cpsievert Jun 21, 2021
cda64ad
patch plotly.js source to support phantomjs/shinytest
cpsievert Jun 21, 2021
d5982ff
bump mathjax version
cpsievert Jun 21, 2021
f52c111
use npm instead of yarn
cpsievert Jun 21, 2021
cd33241
make geom_bar() default to textposition='none'; approve new vdiffr 0.…
cpsievert Jun 21, 2021
32f491b
Migrate to vdiffr 1.0
cpsievert Jun 21, 2021
3435fbb
Use layout.legend.title over layout.annotations to when converting gu…
cpsievert Jun 21, 2021
d541471
wip migrate to gha
cpsievert Jun 21, 2021
1216874
try npm install
cpsievert Jun 21, 2021
b0f7d16
setup node
cpsievert Jun 21, 2021
e621db1
use recommended node version
cpsievert Jun 21, 2021
a12ad8a
try mac instead
cpsievert Jun 21, 2021
ac249b8
install phantomjs; various cleanup
cpsievert Jun 21, 2021
057fe1f
intentionally break a visual test
cpsievert Jun 21, 2021
88d3c45
fix
cpsievert Jun 21, 2021
38e1bdd
use thematic's approach to testing
cpsievert Jun 21, 2021
177935b
bump version
cpsievert Jun 21, 2021
b056974
Merge branch 'master' into plotlyjs-2.0
cpsievert Jun 21, 2021
bce7053
regenerate snapshots
cpsievert Jun 21, 2021
d37f405
map secrets to env variables
cpsievert Jun 21, 2021
af624bb
accept new gha baselines
cpsievert Jun 21, 2021
f1c39a1
whoops
cpsievert Jun 21, 2021
8de5521
hmm
cpsievert Jun 21, 2021
eed311f
approve shinytest baseline
cpsievert Jun 21, 2021
362b6a4
intentionally break a visual test
cpsievert Jun 21, 2021
6b563bf
always upload
cpsievert Jun 21, 2021
ec40dd8
revert change; new baselines
cpsievert Jun 21, 2021
bd63208
Remove the vdiffr dependency and use testthat snaphots directly
cpsievert Jun 21, 2021
8bd4613
More obvious env var naming
cpsievert Jun 21, 2021
e2fae51
more gha details
cpsievert Jun 21, 2021
be22619
always upload
cpsievert Jun 21, 2021
aa44563
new snaps
cpsievert Jun 21, 2021
a4e306c
intentionally break a visual test (again)
cpsievert Jun 21, 2021
e7c86bb
Revert "intentionally break a visual test (again)"
cpsievert Jun 21, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
more gha details
  • Loading branch information
cpsievert committed Jun 21, 2021
commit e2fae519917f286f4debc8ea4a46ececeabc51cf
6 changes: 4 additions & 2 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ jobs:
run: |
pak::pak("shinytest")
shinytest::installDependencies()
pak::pak()
shell: Rscript {0}

- name: Install dependencies
Expand All @@ -111,8 +112,9 @@ jobs:
# Run test() before R CMD check since, for some reason, rcmdcheck::rcmdcheck() skips vdiffr tests
- name: Run Tests
run: |
if (!require(devtools)) install.packages("devtools")
devtools::install()
options(crayon.enabled = TRUE)
if (!require(devtools)) pak::pak("devtools")
if (!require(reshape2)) pak::pak("reshape2")
res <- devtools::test()
df <- as.data.frame(res)
if (sum(df$failed) > 0 || any(df$error)) stop("GHA CI tests failed")
Expand Down