diff --git a/CODEOWNERS b/CODEOWNERS deleted file mode 100644 index 0bc3b70..0000000 --- a/CODEOWNERS +++ /dev/null @@ -1,12 +0,0 @@ -# JR - This defines the maintainer(s) of a repo or the people responsible for different parts thereof. -# The below line will make csgillespie the owner unless it is overriden with the actual owner. - -* @shane82 @dataccino @statsrhian @csgillespie -# More examples: -# https://docs.gitlab.com/ee/user/project/code_owners.html#example-codeowners-file -# -# All files of a type: -# *.Rmd @someone-who-likes-rmd -# -# All files in a directory (recursively): -# /docs/ @all-docs @another-person diff --git a/DESCRIPTION b/DESCRIPTION index 3a5953d..d09fa96 100644 --- a/DESCRIPTION +++ b/DESCRIPTION @@ -1,7 +1,7 @@ Type: Package Package: audit.workbench Title: RStudio/Workbench User Acceptance Tests -Version: 0.6.6 +Version: 0.6.7 Authors@R: person("Jumping", "Rivers", , "info@jumpingrivers.com", role = c("aut", "cre")) Description: Testing whether data scientists can do what they expect in @@ -9,11 +9,12 @@ Description: Testing whether data scientists can do what they expect in License: file LICENSE URL: https://jumpingrivers.github.io/audit.workbench/ Imports: - audit.base (>= 0.6.20), + audit.base (>= 0.6.25), BiocManager, cli, dplyr, fs, + glue, httr, processx, purrr, @@ -21,7 +22,6 @@ Imports: remotes, rlang, rmarkdown, - serverHeaders, stringr, tibble, utils, @@ -33,8 +33,7 @@ Suggests: rvest, testthat (>= 3.0.0) Remotes: - jumpingrivers/audit.base, - jumpingrivers/serverheaders + jumpingrivers/audit.base Config/testthat/edition: 3 Encoding: UTF-8 Language: en-GB diff --git a/NEWS.md b/NEWS.md index 386877e..9ca1ebe 100644 --- a/NEWS.md +++ b/NEWS.md @@ -1,3 +1,9 @@ +# audit.workbench 0.6.7 *2026-02-23* +- fix: Use remote versions of software if possible +- fix: Printing drivers to screen +- fix: Suppress some BioConductor noise +- fix: Check cran didn't work + # audit.workbench 0.6.6 *2026-01-22* - fix: Parsing Posit versions #20 - chore: Linting and air'ing diff --git a/R/check_package.R b/R/check_package.R index b2e20bc..53876b3 100644 --- a/R/check_package.R +++ b/R/check_package.R @@ -14,7 +14,7 @@ check_cran = R6::R6Class( #' @param debug_level See check() for details check = function(debug_level) { private$checker(testing_cran(debug_level)) - rinvisible(NULL) + invisible(NULL) } ), private = list( @@ -93,7 +93,7 @@ testing_bioconductor = function(debug_level) { # Can we access the bioconductor repos # Repo 5 is bioconductor, but doesn't work! # Installing a bioconductor pkg is slowwww - repos = BiocManager::repositories()[1:4] + repos = suppressMessages(BiocManager::repositories()[1:4]) lapply(repos, function(repo) { stopifnot("Unable to access bioconductor repo" = !httr::http_error(repo)) }) diff --git a/R/check_posit_drivers.R b/R/check_posit_drivers.R index b202485..7ce2e4b 100644 --- a/R/check_posit_drivers.R +++ b/R/check_posit_drivers.R @@ -10,7 +10,7 @@ check_posit_drivers = function(debug_level) { upgrade = upgrade, version = latest_version ) - audit.base::print_colour_versions(installed) + print_colour_version_posit_driver(installed[1, ]) installed } @@ -31,3 +31,27 @@ get_installed_posit_driver = function() { } installed_version } + +# fmt: skip +print_colour_version_posit_driver = function(row) { # nolint + software_name = glue::glue("{stringr::str_to_title(row$software)}") # nolint + latest_version = glue::glue("{row$version}") # nolint + if (is.na(row$installed_version)) { + cli::cli_alert_danger( + "{software_name}: latest {latest_version} not installed" + ) + return(invisible(NULL)) + } + + if (isTRUE(row$upgrade)) { + cli::cli_alert_danger( + "{software_name}: v{row$installed_version} installed, \\ + but {latest_version} available" + ) + } else { + cli::cli_alert_success( + "{software_name}: Latest version installed" + ) + } + invisible(NULL) +} diff --git a/R/check_versions.R b/R/check_versions.R index 69e8d2c..33e6ef9 100644 --- a/R/check_versions.R +++ b/R/check_versions.R @@ -19,5 +19,5 @@ check_versions = function(debug_level) { ) installed = tibble::tibble(installed_version = versions, software) - audit.base::augment_installed(installed) + audit.base::augment_installed(installed, remote = TRUE) }