Closed toumban1 closed 11 months ago
Let's make sure we are doing the right thing with the large image files. We may want to gitignore, but look at snapshot test that docs to make sure best practices.
I'm adding an eval statement to the plots tests and the .gitignore will have .svg on the list.
What changes are proposed in this pull request? Unit tests were added for the gnomer-color-palette functions and plotting functions in mutation_viz.
If there is an GitHub issue associated with this pull request, please provide link.
144
Reviewer Checklist (if item does not apply, mark is as complete)
_pkgdown.yml
pkgdown::build_site()
. Check the R console for errors, and review the rendered website.withr::with_envvar(new = c("NOT_CRAN" = "true"), covr::report())
. Begin in a fresh R session without any packages loaded.usethis::use_spell_check()
runs with no spelling errors in documentationWhen the branch is ready to be merged into master:
NEWS.md
with the changes from this pull request under the heading "# cbioportalR (development version)
". If there is an issue associated with the pull request, reference it in parentheses at the end update (seeNEWS.md
for examples).codemetar::write_codemeta()
usethis::use_spell_check()
again