From 0757ad0f4a4814254a39634ba081d27eb122d96f Mon Sep 17 00:00:00 2001 From: Bart Date: Mon, 3 Aug 2026 14:21:42 +0200 Subject: [PATCH 1/5] make error messages runable --- R/get_pvol_nl.R | 4 ++-- R/secrets.R | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/R/get_pvol_nl.R b/R/get_pvol_nl.R index 9dfbe45..adec387 100644 --- a/R/get_pvol_nl.R +++ b/R/get_pvol_nl.R @@ -67,7 +67,7 @@ get_pvol_nl <- function(radar, time, ..., call = rlang::caller_env()) { "There was an authorization error. You may have used an invalid API key.", "i" = "Please check if you set the correct {.val nl_api_key} with - {.code get_secret(\"nl_api_key\")}." + {.run getRad::get_secret(\"nl_api_key\")}." ), cnd = cnd, class = "getRad_error_get_pvol_nl_authorization_failure", @@ -80,7 +80,7 @@ get_pvol_nl <- function(radar, time, ..., call = rlang::caller_env()) { "There was a rate limitation error (HTTP 429) while getting data from the Netherlands.", "i" = "This frequently occurs when using the anonymous key, if it occurs repeatedly consider requesting a registered key ({.url https://developer.dataplatform.knmi.nl/open-data-api#token}). ", "i" = "You can check the current value of {.val nl_api_key} with - {.code get_secret(\"nl_api_key\")}." + {.run getrad::get_secret(\"nl_api_key\")}." ), cnd = cnd, class = "getRad_error_get_pvol_nl_429_failure", diff --git a/R/secrets.R b/R/secrets.R index 6675beb..a2dab8e 100644 --- a/R/secrets.R +++ b/R/secrets.R @@ -79,7 +79,7 @@ get_secret <- function(name) { cli::cli_abort( c( "Can't find secret {.arg {sname}} in the keyring.", - "i" = "Please use {.code set_secret(\"{name}\")} to store the secret. Note that the prefix is automatically added in {.fun set_secret}." + "i" = "Please use {.run getRad::set_secret(\"{name}\")} to store the secret. Note that the prefix is automatically added in {.fun set_secret}." ), class = "getRad_error_secret_not_found" ) From 343822ab346dd90d3b2cdffaca149e9dcb1aec74 Mon Sep 17 00:00:00 2001 From: Bart Date: Mon, 3 Aug 2026 21:57:25 +0200 Subject: [PATCH 2/5] nicer print --- R/get_pvol_nl.R | 4 ++-- R/secrets.R | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/R/get_pvol_nl.R b/R/get_pvol_nl.R index adec387..6d65b97 100644 --- a/R/get_pvol_nl.R +++ b/R/get_pvol_nl.R @@ -67,7 +67,7 @@ get_pvol_nl <- function(radar, time, ..., call = rlang::caller_env()) { "There was an authorization error. You may have used an invalid API key.", "i" = "Please check if you set the correct {.val nl_api_key} with - {.run getRad::get_secret(\"nl_api_key\")}." + {.run [get_secret(\"nl_api_key\")](getRad::get_secret(\"nl_api_key\"))}." ), cnd = cnd, class = "getRad_error_get_pvol_nl_authorization_failure", @@ -80,7 +80,7 @@ get_pvol_nl <- function(radar, time, ..., call = rlang::caller_env()) { "There was a rate limitation error (HTTP 429) while getting data from the Netherlands.", "i" = "This frequently occurs when using the anonymous key, if it occurs repeatedly consider requesting a registered key ({.url https://developer.dataplatform.knmi.nl/open-data-api#token}). ", "i" = "You can check the current value of {.val nl_api_key} with - {.run getrad::get_secret(\"nl_api_key\")}." + {.run [get_secret(\"nl_api_key\")](getrad::get_secret(\"nl_api_key\"))}." ), cnd = cnd, class = "getRad_error_get_pvol_nl_429_failure", diff --git a/R/secrets.R b/R/secrets.R index a2dab8e..a617b9e 100644 --- a/R/secrets.R +++ b/R/secrets.R @@ -79,7 +79,7 @@ get_secret <- function(name) { cli::cli_abort( c( "Can't find secret {.arg {sname}} in the keyring.", - "i" = "Please use {.run getRad::set_secret(\"{name}\")} to store the secret. Note that the prefix is automatically added in {.fun set_secret}." + "i" = "Please use {.run [set_secret(\"{name}\")](getRad::set_secret(\"{name}\"))} to store the secret. Note that the prefix is automatically added in {.fun set_secret}." ), class = "getRad_error_secret_not_found" ) From 39b415541a0891f0dbaf2d5cc5a966eab991dcf5 Mon Sep 17 00:00:00 2001 From: Bart Date: Wed, 5 Aug 2026 08:55:25 +0200 Subject: [PATCH 3/5] do not use rspm as it causes issues on bioconductor --- .github/workflows/R-CMD-check.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/R-CMD-check.yaml b/.github/workflows/R-CMD-check.yaml index 2e08859..56c02e7 100644 --- a/.github/workflows/R-CMD-check.yaml +++ b/.github/workflows/R-CMD-check.yaml @@ -42,7 +42,7 @@ jobs: with: r-version: ${{ matrix.config.r }} http-user-agent: ${{ matrix.config.http-user-agent }} - use-public-rspm: true + use-public-rspm: false - uses: r-lib/actions/setup-r-dependencies@v2 with: From 78be2b0462b69bf9a27030c0024ca399fab6c174 Mon Sep 17 00:00:00 2001 From: bart1 <1662852+bart1@users.noreply.github.com> Date: Fri, 7 Aug 2026 10:33:16 +0200 Subject: [PATCH 4/5] Enable use of public RSPM in workflow revert change --- .github/workflows/R-CMD-check.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/R-CMD-check.yaml b/.github/workflows/R-CMD-check.yaml index 56c02e7..2e08859 100644 --- a/.github/workflows/R-CMD-check.yaml +++ b/.github/workflows/R-CMD-check.yaml @@ -42,7 +42,7 @@ jobs: with: r-version: ${{ matrix.config.r }} http-user-agent: ${{ matrix.config.http-user-agent }} - use-public-rspm: false + use-public-rspm: true - uses: r-lib/actions/setup-r-dependencies@v2 with: From 92ff1ef5e9ed2f22724c9bab757f50b5bdea6a7a Mon Sep 17 00:00:00 2001 From: bart1 <1662852+bart1@users.noreply.github.com> Date: Fri, 7 Aug 2026 10:39:01 +0200 Subject: [PATCH 5/5] Comment out R version 4.1.0 in workflow Comment out the R version 4.1.0 due to installation issue. --- .github/workflows/R-CMD-check.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/R-CMD-check.yaml b/.github/workflows/R-CMD-check.yaml index 2e08859..4112258 100644 --- a/.github/workflows/R-CMD-check.yaml +++ b/.github/workflows/R-CMD-check.yaml @@ -26,7 +26,8 @@ jobs: - {os: ubuntu-latest, r: 'release'} - {os: ubuntu-latest, r: 'oldrel-1'} # And minimum supported version in DESCRIPTION - - {os: ubuntu-latest, r: '4.1.0'} + # this seems to be failing due to a issue installing rhdf5 from bioconductor + # - {os: ubuntu-latest, r: '4.1.0'} env: getRad_nl_api_key: ${{ secrets.GETRAD_NL_API_KEY }}